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

feat(argo-applicationset): update appVersion to 0.4.1 #1223

Merged
merged 4 commits into from
Apr 22, 2022

Conversation

pburgisser
Copy link
Contributor

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:

  • I have bumped the chart version according to versioning
  • I have updated the documentation according to documentation
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.
  • My build is green (troubleshooting builds).

Changes are automatically published when merged to master. They are not published on branches.

Signed-off-by: Philippe <philippe.burgisser@camptocamp.com>
@pburgisser pburgisser force-pushed the update_applicationset_to_041 branch from 161751b to 28aaf86 Compare April 13, 2022 12:33
@pburgisser pburgisser changed the title feat(argo-workflows): update appVersion to 0.4.1 feat(argo-applicationset): update appVersion to 0.4.1 Apr 13, 2022
Signed-off-by: Philippe <philippe.burgisser@camptocamp.com>
Copy link
Member

@mkilchhofer mkilchhofer left a 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

@pburgisser
Copy link
Contributor Author

Indeed, I realized that when I tested it in production with an old version of the Argo-cd helm chart :)

@@ -33,6 +33,7 @@ spec:
securityContext:
{{- toYaml .Values.securityContext | nindent 12 }}
command:
- entrypoint.sh
Copy link
Member

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

@mkilchhofer mkilchhofer merged commit 608e396 into argoproj:master Apr 22, 2022
terrych0u pushed a commit to terrych0u/argo-helm that referenced this pull request Aug 4, 2022
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants