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
Turn off apiserver deprecated metrics #83837
Turn off apiserver deprecated metrics #83837
Conversation
/retest |
Sorry to bother you. Maybe this triggered a potential bug, I need to investigate more. |
/assign @logicalhan |
Rely on #84135. Let's continue after that. |
e180f2d
to
1f8c07e
Compare
/retest |
Retry after kubernetes/perf-tests#893 (That might not help for this PR, but let's try again) /test pull-kubernetes-e2e-gce-100-performance |
/test pull-kubernetes-kubemark-e2e-gce-big |
/hold cancel @kubernetes/sig-api-machinery-pr-reviews |
/cc @krzysied @BenTheElder for e2e framework. |
Maybe @liggitt can help put it in v1.17. :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e2e framework part seems good for me.
/approve
// TODO(krzysied): apiserver_request_latencies_summary is a deprecated metric. | ||
// It should be replaced with new metric. | ||
"apiserver_request_latencies_summary", | ||
"apiserver_request_latency_seconds", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seems good for me because of
apiserver_request_latencies_summary -> apiserver_request_latency_seconds
as https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.14.md#deprecated-metrics
/cc @oomichi |
I will approve if you can get a lgtm from Frederic or another instrumentation reviewer |
/lgtm |
/lgtm |
@smarterclayton Do you have what you need to approve here? |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: oomichi, RainbowMango, smarterclayton The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Turn off metrics that have been deprecated.
Refer to kubernetes/enhancements#1206:
Which issue(s) this PR fixes:
Part of kubernetes/enhancements#1206
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: