-
Notifications
You must be signed in to change notification settings - Fork 2k
feat(argo-cd): Upgrade Argo CD to 2.6.0 #1758
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
Conversation
9f470f0 to
f8130e1
Compare
3fc61e7 to
fa2f120
Compare
909d7bf to
1b55fac
Compare
b6326d1 to
bf46e1e
Compare
32c5afd to
cc41f6a
Compare
Signed-off-by: Petr Drastil <[email protected]>
cc41f6a to
61eb7e7
Compare
jmeridth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
|
Hey everyone, any updates on when this will be released? |
|
@mbastawisy-careem once we get another approval. Thank you for your patience. |
yu-croco
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
| type: object | ||
| required: | ||
| - metadata | ||
| - spec | ||
| type: object | ||
| required: | ||
| - pathParamPrefix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it intentional that this is required now? I'm getting the following error when upgrading:
failed to apply ApplicationSet.argoproj.io, argocd/skip-apps: ApplicationSet.argoproj.io "skip-apps" is invalid: spec.generators[0].git. pathParamPrefix: Required value
The documentation seems to infer that it should be optional:
https://argo-cd.readthedocs.io/en/release-2.6/operator-manual/applicationset/Generators-Git/#git-generator-directories
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upgrade to 2.6.0 stable version
Upgrade notes: https://argo-cd.readthedocs.io/en/latest/operator-manual/upgrading/2.5-2.6/
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
Changes are automatically published when merged to
main. They are not published on branches.