-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Helm Chart app.labels are not applied to all deployed resources #9158
Comments
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
#9490 has addressed this problem to a large degree, except for subcharts. |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close not-planned |
@k8s-triage-robot: Closing this issue, marking it as "Not Planned". In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
What happened?
The Helm Chart value
app.labels
is described as:But it's only applied to
ServiceMonitor
,NetworkPolicy
, andPodDisruptionBudget
What did you expect to happen?
The Labels should get applied to all Deployments, Services, ConfigMaps and arguably even Resources deployed by the Subcharts.
How can we reproduce it (as minimally and precisely as possible)?
Template the Helm Chart with the following values snippet
and look at the resulting manifests.
Anything else we need to know?
An easy fix would be to include
.Values.app.labels
intokubernetes-dashboard.labels
heredashboard/charts/kubernetes-dashboard/templates/_helpers.tpl
Line 49 in ef603b2
and removing the 3 occurrences where
.Values.app.labels
are used directly.If the labels should be propagated to the subcharts (which I think they should) that could be also easily done in
values.yaml
I'd be happy to submit a PR for this if you agree with the solution.
Thank you in advance
What browsers are you seeing the problem on?
No response
Kubernetes Dashboard version
7.5.0
Kubernetes version
any
Dev environment
No response
The text was updated successfully, but these errors were encountered: