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
feature(NodeAffinity): return Skip in PreScore when nothing to do in Score #117024
feature(NodeAffinity): return Skip in PreScore when nothing to do in Score #117024
Conversation
Please note that we're already in Test Freeze for the Fast forwards are scheduled to happen every 6 hours, whereas the most recent run was: Thu Mar 30 22:31:05 UTC 2023. |
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. |
040bd75
to
7fd2c6d
Compare
/retest |
pkg/scheduler/framework/plugins/nodeaffinity/node_affinity_test.go
Outdated
Show resolved
Hide resolved
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.
sorry for missing this one, just some suggestion otherwise LGTM.
pkg/scheduler/framework/plugins/nodeaffinity/node_affinity_test.go
Outdated
Show resolved
Hide resolved
56920fd
to
9f1e29c
Compare
OK, there is a bug on the framework side... 😓 |
Given the bug happens in an extremely minor case, I make a change in e2e rather than changing the framework side. |
d294a3d
to
c337b63
Compare
/assign |
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
pkg/scheduler/framework/plugins/nodeaffinity/node_affinity_test.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/nodeaffinity/node_affinity_test.go
Outdated
Show resolved
Hide resolved
pkg/scheduler/framework/plugins/nodeaffinity/node_affinity_test.go
Outdated
Show resolved
Hide resolved
LGTM label has been added. Git tree hash: f74a13eae7cec1e6111e9421420444c65fa2a872
|
/hold It's better to go thru the approver's review as well. |
5bdf326
to
66a4329
Compare
Fixed @chendave 's point. @kubernetes/sig-scheduling-approvers Please anyone take a look for |
66a4329
to
1d04531
Compare
what do you mean by this? |
/lgtm |
LGTM label has been added. Git tree hash: 43df7748a53e7c3481858c5a7ab02b9d2bdbe7a8
|
In metrics, after this PR is merged, it probably looks like NodeAffinity's Score plugin's latency go increasing because we execute NodeAffinity Score plugin only when it calculates something. |
Ok, that's not very clear in the release notes. Maybe reword like this:
|
@alculquicondor Updated. |
/lgtm |
/hold cancel |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR make NodeAffinity PreScore returning Skip when nothing to do in NodeAffinity Score
Which issue(s) this PR fixes:
part of #115745
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: