Skip to content

Conversation

@abdennour
Copy link

ArgoCD uses Helm just as YAML generator.
Custom ArgoCD plugins ignore helm hooks as well.

More details:
https://argo-cd.readthedocs.io/en/stable/user-guide/helm/

Signed-off-by: abdennour

Signed-off-by: abdennour <mail@abdennoor.com>
@abdennour
Copy link
Author

@shehroz please FYA

@abdennour
Copy link
Author

@pcentrella FYI please

@bd-skatzman
Copy link

Thank you for the PR. I understand the utility for ArgoCD, but this adds specificity that doesn't apply for all use cases, customers, and scenarios. It could, however, be advantageous to provide a some form of generic annotations definition in the helper to allow customers to define annotations to meet their specific use case scenarios such as ArgoCD integration.

@abdennour
Copy link
Author

@skatzman-synopsys thank you for sparing time to review the PR. I do partially agree. Accordingly, we can add flag like .Values.managedByArgoCD and if it's "true", we should inject those annotations, instead of having it hard-coded as now.
Please let me know if this work, so i can update PR accordingly

@bd-skatzman
Copy link

Filed internal issue HUB-32224 to prioritize and track this change request. This pull request should remain open in alignment with HUB-32224 resolution.

@qerub
Copy link

qerub commented Apr 25, 2025

Argo CD has support for most Helm hooks nowadays: https://argo-cd.readthedocs.io/en/latest/user-guide/helm/#helm-hooks

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.

3 participants