-
Notifications
You must be signed in to change notification settings - Fork 469
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
Add ability to set your own common labels for all resources #814
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a description to this PR describing what it changes and what problem it solves.
apologies for missing out the description, I've updated it now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the new field to the json schema file. Also, please bump the chart's patch version and then run make generate-examples CHARTS=opentelemetry-collector
.
@@ -4,7 +4,7 @@ | |||
|
|||
nameOverride: "" | |||
fullnameOverride: "" | |||
|
|||
commonLabels: {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a description the explains this field similar to https://github.com/prometheus-community/helm-charts/blob/8b91a619e9407c0e830df7071ee73cd160937b6b/charts/kube-prometheus-stack/values.yaml#L25-L27.
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
This change gives us a new optional variable that we can set which allows you to add your own custom labels to all of the resources being deployed. Currently we are able to do this with Pods but not anything else like deployments, config-maps etc.
This is already quite common among many other open source helm charts. It fixes a problem for us where we use these labels for identifying which teams own certain resources which in turn helps us direct prometheus alerts to the correct teams.