-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
feat(argo-applicationset): update appVersion to 0.4.1 #1223
feat(argo-applicationset): update appVersion to 0.4.1 #1223
Conversation
Signed-off-by: Philippe <philippe.burgisser@camptocamp.com>
161751b
to
28aaf86
Compare
Signed-off-by: Philippe <philippe.burgisser@camptocamp.com>
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.
Its a bit in conflict with #1219. Maybe after we merge this change here, we deprecated the charts directly and only maintain it in argo-cd chart directly
Indeed, I realized that when I tested it in production with an old version of the Argo-cd helm chart :) |
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
@@ -33,6 +33,7 @@ spec: | |||
securityContext: | |||
{{- toYaml .Values.securityContext | nindent 12 }} | |||
command: | |||
- entrypoint.sh |
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.
Synced this change also from the upstream manifests
* feat: update appVersion to 0.4.1 Signed-off-by: Philippe <philippe.burgisser@camptocamp.com> * Update changelog Signed-off-by: Philippe <philippe.burgisser@camptocamp.com> * fix: Also sync entrypoint.sh change from upstream Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com> Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
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
master
. They are not published on branches.