Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrated ArgoCD plugin parameters support ? #50

Closed
christiansen-merel opened this issue Jan 19, 2024 · 3 comments
Closed

Integrated ArgoCD plugin parameters support ? #50

christiansen-merel opened this issue Jan 19, 2024 · 3 comments

Comments

@christiansen-merel
Copy link

christiansen-merel commented Jan 19, 2024

Hello.
First, thanks for the great work.

Is there a plan to support plugin parameters in Application scope ? here in documentation

For the example case from the ArgocD documentation:

apiVersion: argoproj.io/v1alpha1
kind: Application
spec:
  source:
    plugin:
      parameters:
        - name: FOO
          value: bar
        - name: REV
          value: test-$ARGOCD_APP_REVISION

The plugin would add --state-values-set FOO=bar --state-values-set REV=test-$ARGOCD_APP_REVISION to the helmfile command.

It could also use --state-values-file if the parameter value appears to be a file. It could fix issue #46 if I understand it well.

Does it makes sense ?

I might submit a PR if interrested.

@christiansen-merel
Copy link
Author

christiansen-merel commented Jan 25, 2024

Juste read the CMP Proposal I previously missed.

I guess it could be better to add values parameter(s) that mimic the helmfile release.values parameter.
This feature would be valuable for ApplicationSet especially.

Usage example:

apiVersion: argoproj.io/v1alpha1
kind: Application
spec:
  source:
...
    plugin:
      parameters:
        - name: values
          map:
            attributeOne: foo
            attributeTwo: bar
          array:
          - values-as-file.yaml

@travisghansen
Copy link
Owner

I'm open to different approaches here for sure. I do support parameters already for the env vars that already exist, nothing preventing us from adding more features on this front.

@travisghansen
Copy link
Owner

See the parameters sub-command in the script around line 466. That are all available as cmp params and gui.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants