-
Notifications
You must be signed in to change notification settings - Fork 39
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
Support for hooks and Helm hooks #149
Comments
Good news, I just found that ArgoCD support helm hooks natively. |
@kvaps I'm curious about what hooks need to be supported in qbec with some examples. |
Sure! Most of the examples can be found in helm charts/stable 61 chart of 282 are using helm hooks:
Let's take stolon (high available postgress cluster):
Another good example is wire-server charts:
|
I'm so sad that Kubernetes API does not allow to overwrite the Job manifests and execute the jobs according the changed hash of podTemplate for them, that's why all these hooks are needed for. |
@gotwarlost it seems all the needed code for hooks is already implemented in gitops-engine project: This project combines the best from Argo CD and Flux CD. The only problem of ArgoCD is no opportunity to apply changes locally (in case if something has gonna wrong). |
It would be nice to expand qbec directives with hooks similar to which helm already has:
https://helm.sh/docs/topics/charts_hooks/#the-available-hooks
Also to translate helm hooks the same way in expandHelmTemplate
The text was updated successfully, but these errors were encountered: