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

Fix ArgoCD hooks annotation location #3989

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

gcaracuel
Copy link
Contributor

@gcaracuel gcaracuel commented May 13, 2024

Changes proposed in this PR

ArgoCD hooks to be set to Job annotations and not at Job template spec level -> https://argo-cd.readthedocs.io/en/stable/user-guide/resource_hooks/#usage

If set at current Job template spec level when Job is finished ArgoCD detects back again diff entering into a loop, if ArgoCD AutoSync is enabled this will be a loop of init jobs running forever.

fix: #3831

How I've tested this PR

  • Ran helm bats tests successfully.

How I expect reviewers to test this PR

  • Deploy this chart using ArgoCD Application. Without the fix after the sync it will enter in a diff sync loop.

Checklist

Copy link

hashicorp-cla-app bot commented May 13, 2024

CLA assistant check
All committers have signed the CLA.

@gcaracuel
Copy link
Contributor Author

Am I missing anything for this to be reviewed?

Copy link
Contributor

@ndhanushkodi ndhanushkodi left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this! Waiting for a test run and then I can officially approve. If in the meanwhile you don't mind making these small changes to the changelog that'd be great!

.changelog/3989.txt Outdated Show resolved Hide resolved
@ndhanushkodi ndhanushkodi added the pr/no-backport signals that a PR will not contain a backport label label Jun 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.3.x backport/1.4.x backport/1.5.x pr/no-backport signals that a PR will not contain a backport label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ArgoCD hook annotations on the server-acl-init-job do not work
2 participants