-
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 HelmRelease CRD #5984
Comments
|
|
The spec is a little overwhelming for me to be sure, but is this possibly very simple? e.g. is there a 1:1 mapping between the HelmRelease CRD and a Helm chart?And then just look in |
Yes, you could probably try and make sense using schema. But in simple terms, 3 fields that we need to be worried are here :
Not sure if this is of any help. |
Can you create a demo public repo that contains an example HelmRelease CRD that you know can be updated? i.e. to test any solution against |
And here is the repo setup for test : https://github.com/adusumillipraveen/demo-flux-config/blob/master/demo-flux.yaml |
It's probably worth emphasising here; with the HelmRelease CRD, it's possible to specify either a chart registry, or a git repo with a path to a chart. |
Thanks @ChipWolf , will give it a try now and let you know how it goes. |
That works perfect, Thanks @ChipWolf . Appreciate your time on this. |
Per #7415, would love to see both Helm Operator and Helm Controller support. Helm Controller is the eventual successor of Helm Operator. It has additional complexity as it breaks the Helm Repository/Git Repository portion into it's own CRD. So an implementation would need to a two-pass approach to accumulate repository urls and their labels/name, and then accumulate |
For those interested, I and @bjw-s created a Github action (and standalone script) to update the Helm Release files with a renovate annotation (comment). It expects your See here: https://github.com/k8s-at-home/renovate-helm-releases |
I've taken a crack at implementing a proper Flux package manager that can renovate |
Feel free to give my branch a try on your Flux repositories. The local development instructions make it fairly simple. I've tested my changes on my main Flux repository and it is working well so far. Note that you will need to define |
@danports This is awesome! I'll try to get some time in to test against my repository as well. Also, there's a whole bunch of repos you could pull down and test this on over at https://github.com/k8s-at-home/awesome-home-kubernetes |
To sum up the testing steps from the docs...
|
@onedr0p Yep - you can omit |
It worked fine for me, this is amazing. Can't wait to see it merged, thanks @danports |
🎉 This issue has been resolved in version 31.35.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
updating helmreleases is now supported upstream, just waiting on gitrepos renovatebot/renovate#5984 https://github.com/renovatebot/renovate/releases/tag/31.35.0
What would you like Renovate to be able to do?
Raise Pull Requests for updates to HelmRelease CRDs with latest helm charts versions.
Describe the solution you'd like
Extend the solution of Helm requirements.yaml to renovate HelmRelease CRD. For example, actual version that needs to be updated is here
Describe alternatives you've considered
No other alternatives available at the moment.
Additional context
Reuse as described in #3414 (comment)
The text was updated successfully, but these errors were encountered: