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 scheduler extenders being not called on preemptions #103019
Fix scheduler extenders being not called on preemptions #103019
Conversation
Welcome @ordovicia! |
Hi @ordovicia. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
/ok-to-test |
/lgtm |
Can you please create a cherry pick to 1.21 and 1.20 once this is merged? |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahg-g, ordovicia 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 |
Thanks, it would be great if you can add an integration test to https://github.com/kubernetes/kubernetes/blob/master/test/integration/scheduler/extender_test.go |
@ordovicia I'm interested in adding this integration test, may I take it? |
pls go ahead. |
Thank you! I will start working on that. |
The unit test ensures extenders are being called during preemption process as per changes in PR kubernetes#103019. To ensure extenders are being called during preemption, had to add the following in TestGenericSchedulerWithExtenders: - subtest to register defaultpreemption plugin as PostFilter - runtime.WithExteenders() to ensure extenders are passed in test To register defaultpreemption plugin as PostFilter, had to define RegisterPostFilterPlugin in pkg/scheduler/testing/framework_helpers.go See also: kubernetes#103019
/triage accepted |
…03019-upstream-release-1.21 Automated cherry pick of #103019: Fix frameworkImpl.extenders being not set
…03019-upstream-release-1.20 Automated cherry pick of #103019: Fix frameworkImpl.extenders being not set
@ordovicia @ahg-g I apologize for the long absence. I originally working on adding an integration test, then I discussed with Huang-Wei about covering this with a unit test instead; however, other things have since come up, and I don't think I will get to this in a reasonable amount of time. I closed my PR to avoid holding things up any further. I believe a test is still needed. |
What type of PR is this?
/kind bug
/sig scheduling
What this PR does / why we need it:
This PR fixes a bug that scheduler extenders are not called on preemptions because
frameworkImpl.extenders
is not set.Which issue(s) this PR fixes:
Fixes #102971
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: