-
Notifications
You must be signed in to change notification settings - Fork 39.4k
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
Tracking nodes with pods with affinity #84824
Tracking nodes with pods with affinity #84824
Conversation
[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 |
/lgtm |
5bf1608
to
f2913bd
Compare
/priority important-soon |
// WithAffinityPodsList lists nodes. | ||
func (nodes NodeInfoLister) HavePodsWithAffinityList() ([]*schedulernodeinfo.NodeInfo, error) { | ||
return nodes, nil | ||
} | ||
|
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.
Comments needs to be updated.
f2913bd
to
950e1a4
Compare
/lgtm |
950e1a4
to
f83a20c
Compare
f83a20c
to
c33f217
Compare
/retest |
1 similar comment
/retest |
/lgtm |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Adds a list to nodeinfo snapshot that tracks the nodes with at least one pod with affinity terms. This is useful to avoid iterating over unrelated nodes for inter-pod affinity priority and inter-pod anti-affinity predicate
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
This will help avoid the performance degradation from #84669
Before:
After:
Does this PR introduce a user-facing change?:
/assign @Huang-Wei