Skip to content
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

[prometheus-elasticsearch-exporter] Update pod labels #5031

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jimmybchopps
Copy link

What this PR does / why we need it

To increase traceability throughout the application by adding more of the standardized kubernetes labels to pod workloads in addition to the stateful set itself.

Special notes for your reviewer

Originally, the pod labels were only get the selector labels, this update adds all including the selector labels so no labels are being removed; only a few are being added to the pod.

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Signed-off-by: Jimmy Bourque <[email protected]>
Signed-off-by: Jimmy Bourque <[email protected]>
@jimmybchopps
Copy link
Author

PR ready for review please @svenmueller @desaintmartin @zeritti

Please let me know if there are any questions. Thanks!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant