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

Bump chart to have jobLabel for on prometheus-exporter service #137

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

Conversation

lconsuegra
Copy link
Member

@lconsuegra lconsuegra commented Dec 13, 2024

Description of the changes

A breaking change was introduced in the prometheus-node-exporter chart : commit and was not reported in the values of the umbrella chart.

This version of the chart (62.3.0) fix the problem.

This is not a PR for merge but to avoid removal of the branch that we are temporarily using on our environments and waiti for the upgrade in main branch.

Thanks.

Breaking change

  • No
  • Yes (in the Helm chart(s)): indicate the chart, version & release note link
  • Yes (in the module itself): give an explanation of the breaking change

Tests executed on which distribution(s)

  • KinD
  • AKS (Azure)
  • EKS (AWS)
  • Scaleway
  • SKS (Exoscale)

@lconsuegra lconsuegra self-assigned this Dec 13, 2024
@lconsuegra lconsuegra force-pushed the fix/node_exporter_job_label branch from 43dafee to 3adf262 Compare December 16, 2024 07:41
@lconsuegra lconsuegra changed the title Adapt podLabels values since it was changed in prometheus-node-exporter subchart Bump chart to have jobLabel for on prometheus-exporter service Dec 16, 2024
@lconsuegra lconsuegra force-pushed the fix/node_exporter_job_label branch from 23299f9 to 35eceeb Compare December 16, 2024 07:59
@lconsuegra lconsuegra force-pushed the fix/node_exporter_job_label branch from 35eceeb to d5728d0 Compare December 16, 2024 07:59
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