-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 Renovating helm charts #3414
Comments
+1 for this issue Looks like following code is what checks if it is updating kubernetes manifests or not and skips values.yaml even if it is found by the regex.
Could this check be skipped somehow? or do you reckon a new manager needs to be implemented for this? |
Disclaimer: I don't use kubernetes or helm daily, but it sounded to me like they're different managers even if the syntax is similar. I assume there would need to be other if/else checks and logic differences between k8s and helm too? Cc @mxey who wrote the k8s part originally |
Thanks for your quick response. Another manager sounds good, but I think the code base would be pretty similar. Because yamls in helm charts are templated manifests. Also, I would also like to understand how renovate would handle
|
In Renovate it's pretty common to have multiple package files per repo and multiple dependencies per package file, so Helm would be no different. Each dependency is tracked to a package file, which is identified by file name. The way dependencies within a package file are identified varies per-manager. In this case it's probably done by line number. |
This looks like an awesome feature. Thanks @nchashch for your work! I would also be interested in having support for Helm release CRDs like the flux Helm operator uses: https://docs.fluxcd.io/projects/helm-operator/en/latest/references/helmrelease-custom-resource.html I guess the indexer could be reused from this? |
🎉 This issue has been resolved in version 19.61.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
No description provided.
The text was updated successfully, but these errors were encountered: