[Debezium Operator Helm chart] Conflicting `app.kubernetes.io/managed-by` label
- Dominant language
- HTML
- Stars
- 6
- Forks
- 8
- Avg merge
- 2d 19h
- Merged PRs (30d)
- 1
Description
`debezium-operator` helm charts hardcodes label `app.kubernetes.io/managed-by: quarkus` to resources deployed by chart.
It's problematic, as it goes against Helm best practices. https://docs.helm.sh/docs/chart_best_practices/labels/#standard-labels. It's expected for label `app.kubernetes.io/managed-by` to be set as `{{ .Release.Service }}`. Or just `Helm` if you intend to hardcode it, as this is what effectively a result of rendering the templates. Or just drop it, `helm install` can add it automatically.
With custom label value we see probalematic behaviour when kustomizing and diffing the chart, somewhere during the pipeline expected label `app.kubernetes.io/managed-by: Helm` appears and this triggers a perpetual diff between planned and actual state of labels.
Contributor guide
Assessment
This issue has not been assessed yet.