-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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: Support score extension function in preemption. #121867
Conversation
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/sig scheduling |
/cc @Huang-Wei |
41fc749
to
df92108
Compare
@Huang-Wei Already updated according to the review, PTAL:) |
df92108
to
3994bb0
Compare
/retest |
/lgtm @lianghao208 could you add a release note to this PR:
cc @kubernetes/sig-scheduling-misc for awareness. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Huang-Wei, lianghao208 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 |
LGTM label has been added. Git tree hash: 8c17109396b1cae6b419be8f831641d6e82c4911
|
@Huang-Wei released note added, thanks for the review. |
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.
Just nits.
3994bb0
to
34e620d
Compare
/lgtm as well, thanks! |
/lgtm |
LGTM label has been added. Git tree hash: 33a984742fd5543afd1b254d9e6e899a3084b2ff
|
What type of PR is this?
/kind feature
/sig scheduling
What this PR does / why we need it:
Adding a
PreemptionScoreFuncs()
in preemption.go#Interface to extend the scoreFuncs in preemption.go#pickOneNodeForPreemption() .In some use cases, we need the capability to extend the prioritization for node victims to be preepmted instead of rewriting the whole SelectVictimsOnNode().
Which issue(s) this PR fixes:
discussion: slack discussion
Special notes for your reviewer:
Does this PR introduce a user-facing change?