grafana / oncall

Developer-friendly incident response with brilliant Slack integration
GNU Affero General Public License v3.0
3.55k stars 294 forks source link

fix(helm): Fixed migrate not being consistent in name for hooks #4968

Open elliotcourant opened 3 months ago

elliotcourant commented 3 months ago

What this PR does

This makes it so that the yaml being created for the migrate job is consistent when useHook is true or false. At the moment if use hook is false, the the name on the yaml and in the metadata match. But if use hook is true then the name on the object itself is different from the one in the metadata.

Which issue(s) this PR closes

Didn't open an issue for this since it's minor.

Basically I have an "infra" repo that generates the yaml from several helm charts and version controls the generated yaml. It's an easy way to make sure that a chart update doesn't absolutely clobber something and makes it very easy to see whats actually getting pushed out to the kube cluster before actually applying it.

Because of the way this "metadata.name" field was being generated, it would result in the file being changed each time the yaml for the repository was generated which made for some odd diffs when something completely different may have changed.

Checklist


Not aware of any tests for this specifically in this repo that I can see, or documentation around this specific behavior.

CLAassistant commented 3 months ago

CLA assistant check
All committers have signed the CLA.

github-actions[bot] commented 2 months ago

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

elliotcourant commented 2 months ago

Just looking for feedback still, not stale

github-actions[bot] commented 1 day ago

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!