Skip to content

chore: add helm valuefiles wildcard support examples#465

Open
nitishfy wants to merge 3 commits into
argoproj:masterfrom
nitishfy:helm-wildcard-examples
Open

chore: add helm valuefiles wildcard support examples#465
nitishfy wants to merge 3 commits into
argoproj:masterfrom
nitishfy:helm-wildcard-examples

Conversation

@nitishfy

Copy link
Copy Markdown
Member

Examples for argoproj/argo-cd#26768

Signed-off-by: nitishfy <justnitish06@gmail.com>
Signed-off-by: nitishfy <justnitish06@gmail.com>
Signed-off-by: nitishfy <justnitish06@gmail.com>

@reggie-k reggie-k left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nitishfy

Copy link
Copy Markdown
Member Author

@reggie-k I didn't understand what you mean. We're putting that already in the argocd-example-apps repository.

valueFiles:
# no files will be matched, but since ignoreMissingValueFiles is true, it will not error out
# hence default values will be used for the helm chart
- values/no-match/staging-*.yaml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- values/no-match/staging-*.yaml
- values/no-match/staging-*.yaml
destination:

(The spaces number may be wrong though as suggesting from the GH UI)

@reggie-k

Copy link
Copy Markdown
Member

@reggie-k I didn't understand what you mean. We're putting that already in the argocd-example-apps repository.

Oh SORRY! Getting senile, thought it was at the main repo. Thanks!

@cardoe cardoe left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're missing destination: in app-08.yaml

@agaudreault agaudreault left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow the pattern of existing apps.

This is to provide examples of the features supported by Argo. I don't think it is necessary to have 8 different apps for this feature. Just add one. The other features should be documented and validated as unit tests.

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

Successfully merging this pull request may close these issues.

4 participants