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] Update referenced node-exporter to 4.7.* #2747

Merged
merged 1 commit into from
Nov 28, 2022
Merged

[prometheus] Update referenced node-exporter to 4.7.* #2747

merged 1 commit into from
Nov 28, 2022

Conversation

skyhisi
Copy link
Contributor

@skyhisi skyhisi commented Nov 28, 2022

Signed-off-by: Silas Parker silas@srp.me.uk

What this PR does / why we need it

Update referenced node-exporter to 4.7.*
Allows to use new node-exporter chart 4.7.1 which allows setting environment variables correctly, which should allow the GOMAXPROCS=1 work around described in prometheus/node_exporter#2500 to be applied

Which issue this PR fixes

Special notes for your reviewer

Checklist

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

Signed-off-by: Silas Parker <silas@srp.me.uk>
@monotek monotek merged commit c3ed9d8 into prometheus-community:main Nov 28, 2022
@SuperQ
Copy link
Contributor

SuperQ commented Nov 28, 2022

Thanks, it would also be good to update kube-prometheus-stack with the same version.

Matiasmct pushed a commit to giffgaff/prometheus-charts-backup that referenced this pull request May 16, 2023
…munity#2747)

Signed-off-by: Silas Parker <silas@srp.me.uk>

Signed-off-by: Silas Parker <silas@srp.me.uk>
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.

3 participants