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
Rename pod_preemption_metrics to preemption_metrics. #93256
Conversation
…c's type was changed from Gauge to Histogram, renaming it should make it easier to providers to migrate
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahg-g 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 |
/retest |
/assign @logicalhan |
/lgtm |
/retest |
@Huang-Wei let me know if you have concerns. |
/milestone v1.19 |
/priority important-soon |
LGTM. |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Since this metric's type was changed from Gauge to Histogram, renaming it should make it easier for providers to migrate. Also, the name now matches better with preemption_attempts_total (_total suffix because it is a counter)
Which issue(s) this PR fixes:
Part off #83464
Does this PR introduce a user-facing change?: