gloo icon indicating copy to clipboard operation
gloo copied to clipboard

Add custom label to all objects in kubernetes chart

Open TheCubicleJockey opened this issue 1 year ago • 4 comments

Do you have a suggestion for code improvement or tracking existing technical debt? Please describe.

This chart improvement would allow the user to include a defined label in the value.yaml, and that label would be included in the kubernetes objects in metadata.label where allowed by kubernetes. I can write the PR and it would be a short PR and clear PR.

Describe the solution you'd like

Using something like commonLabels at the top level of the values file, that can be optionally included with all the kubernetes object that allow for labeling.

Additional Context

Labels are good in kubernetes for organization and for some custom tools that read labels as a part of rule sets.

TheCubicleJockey avatar Jan 05 '24 20:01 TheCubicleJockey

Am I good to make the PR?

TheCubicleJockey avatar Jan 17 '24 21:01 TheCubicleJockey

My use case is for the clusterrole to have a label. So, I could make one specifically for the clusterrole, but I figured most charts have a values.commonLabels or a values.additionalLabels as a more generally acceptable version that would fit most need and my needs.

If i am doing the work, I would like to help improve the chart in a way that you or your team might like it improved.

TheCubicleJockey avatar Jan 23 '24 18:01 TheCubicleJockey

PR for this work https://github.com/solo-io/gloo/pull/9195

TheCubicleJockey avatar Feb 26 '24 20:02 TheCubicleJockey