Conversation
|
Mind if I get some feedback on the above before continuing? |
|
Re: the location: one interesting question in this context is how the new Helm chart will relate to the static Kubernetes release manifests we currently maintain below the
I'm slightly leaning towards (1) in order to not make Helm a hard requirement to install CCM, at least not for now. In that vein, it could make sense to move the Re: the initial two variables: that looks good to me. 👍 (The full Helm adoption should probably cover using Helm for the release as well, meaning we'd want to update our Github Actions workflows. That's not something we need to do in this PR though, and it can also be done by someone else.) |
|
Thanks @timoreimann, The placement of the helm chart code tends to be developer preference since the resulting artifact is usually hosted separately from the code repository anyway. I'll go ahead and move the |
|
@vyas-n sounds good, thanks! |
This is a starting point to resolve #629.
Mind if I get some feedback on how this is so far?
I'd like to start extracting some variables in the helm chart and place it into the values.yaml file. Would it be ok if I proceeded with the following variables: