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: Upgrades the prometheus http client to 1.13 #47707

Closed
wants to merge 2 commits into from

Conversation

yesoreyeram
Copy link
Contributor

PR to validate the fix prometheus/client_golang#1022

@bboreham
Copy link
Contributor

bboreham commented May 5, 2022

Results of trying this out - solid lines are incidences of the error, and dotted line is when we rolled out the fix.

Screenshot from 2022-05-05 11-02-47

@bboreham
Copy link
Contributor

Was this change made somewhere else?

@yesoreyeram
Copy link
Contributor Author

@bboreham - The fix was tested in https://github.com/grafana/grafana-private-mirror/commits/v8.5.0-felmo1. Actual fix will be implemented once the promclient released and tested (in a separate PR). This PR was just here to validate the fix.

@yesoreyeram
Copy link
Contributor Author

nvm. Will keep this PR open and update this once the promclient released.

@yesoreyeram yesoreyeram reopened this May 17, 2022
@grafanabot
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@grafanabot grafanabot added the stale Issue with no recent activity label Jun 17, 2022
@bboreham
Copy link
Contributor

I see it's been added to a 1.13.0 milestone, but no date attached.

@grafanabot grafanabot removed the stale Issue with no recent activity label Jun 22, 2022
@grafanabot
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@grafanabot grafanabot added the stale Issue with no recent activity label Jul 22, 2022
@grafanabot
Copy link
Contributor

This pull request has been automatically closed because it has not had activity in the last 2 weeks. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@grafanabot grafanabot closed this Aug 5, 2022
@bboreham
Copy link
Contributor

bboreham commented Aug 8, 2022

Prometheus client library has now been released with the fix.
https://github.com/prometheus/client_golang/releases/tag/v1.13.0

@bboreham bboreham reopened this Aug 8, 2022
@grafanabot grafanabot removed the stale Issue with no recent activity label Aug 9, 2022
Signed-off-by: bergquist <carl.bergquist@gmail.com>
@bergquist bergquist added this to the 9.2.0 milestone Aug 18, 2022
@bergquist bergquist added backport v9.1.x Bot will automatically open backport PR add to changelog labels Aug 18, 2022
@bergquist bergquist changed the title update prom client Prometheus: Upgrades the prometheus http client to 1.13 Aug 18, 2022
@bergquist
Copy link
Contributor

Creating a new PR was faster than resolving merge conflicts :) #53901

@dsotirakis dsotirakis removed this from the 9.2.0 milestone Sep 22, 2022
@dsotirakis dsotirakis added this to the 9.2.0-beta1 milestone Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/backend backport v9.1.x Bot will automatically open backport PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants