grafana / helm-charts

Apache License 2.0
1.67k stars 2.29k forks source link

`managed-by` is set at install time by the managing installer, and should not be templated. #3389

Closed bleggett closed 4 weeks ago

bleggett commented 4 weeks ago

app.kubernetes.io/managed-by: Helm is automatically applied to resources installed by Helm via helm install, and does not need to be (read: should not be) manually templated. The label is an install-time label that should be applied by the tool actually installing the resources into the cluster.

This matters because helm template should (and by default will) produce YAML without this label, as it is unknown at template-time what will "manage" the resource - e.g. helm template | kubectl apply -f - should not produce resources with the app.kubernetes.io/managed-by: Helm label, because the resources are not managed by helm.

xref: https://github.com/istio/istio/issues/53698

CLAassistant commented 4 weeks ago

CLA assistant check
All committers have signed the CLA.

jkroepke commented 4 weeks ago

Hi @bleggett

thanks for you request. Please create a dedicated PR for each chart and increase the version separately.