-
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
feat: Add Pod Scheduling SLI Duration metric #119049
Conversation
/test pull-kubernetes-node-e2e-containerd |
This PR may require stable metrics review. Stable metrics are guaranteed to not change. Please review the documentation for the requirements and lifecycle of stable metrics and ensure that your metrics meet these guidelines. |
Hello @helayoty, Please let me know if I can be of any assistance :) |
LGTM. @helayoty could you squash the commits? |
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.
/label tide/merge-method-squash
/triage accepted |
@helayoty hi, Bug Triage Lead here 👋🏼 Since it was communicated in earlier weeks (3 & 1 weeks before), Code Freeze Deadline passed and, this PR has not made to the current milestone, hence I am moving this PR to the next milestone. /milestone next-candidate |
Signed-off-by: Heba Elayoty <[email protected]> Co-authored-by: Aldo Culquicondor <[email protected]>
@furkatgofurov7 @alculquicondor , just back from vacation. The PR was ready to merge but just needed a squash. I did that now, and it's up to you to decide if we need an exception or if it can wait for the next release. |
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.
/lgtm
LGTM label has been added. Git tree hash: 320cae4c9b98d06bde6efeb983a1b9b69aabd57f
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: helayoty, Huang-Wei, logicalhan 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 |
@helayoty I am fine with both options, although I do not know if we have still time for exception. Tagging 1.28 Release Lead for visibilty /cc @gracenng |
I'm inclined to leave this for the next release |
Add a new scheduler metric,
pod_scheduling_sli_duration_seconds
.What type of PR is this?
/kind feature
/sig scheduling
What this PR does / why we need it:
As a final change to the update related to KEP #3521, discussion can be found here.
Which issue(s) this PR fixes:
NA
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Yes. New metric,
pod_scheduling_sli_duration_seconds
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
KEP #3521