-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Fix prometheus.Handler() deprecation #48664
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fraenkel Assign the PR to them by writing Associated issue: 48297 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
48583f7
to
b46184c
Compare
/unassign |
694e3cb
to
ea1fd3d
Compare
Unit failure due to data race in grpc. Needs a bump to pick up |
/test pull-kubernetes-e2e-kops-aws |
1. Replace deprecated Handler with like function from promhttp 2. Prevent metric naming collisions during tests which load the old handler by overriding the subsystem
82ebc2f
to
f05c1e4
Compare
@fraenkel PR needs rebase |
This PR hasn't been active in 90 days. Closing this PR. Please reopen if you would like to work towards merging this change, if/when the PR is ready for the next round of review. You can add 'keep-open' label to prevent this from happening again, or add a comment to keep it open another 90 days |
The prometheus.Handler() has been deprecated. This replaces the existing handler with one using their new tooling via the promhttp package.
fixes #48297