-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Optimize pod affinity 2 #29379
Optimize pod affinity 2 #29379
Conversation
daf2315
to
88ca042
Compare
88ca042
to
9d9e686
Compare
9d9e686
to
84a24bf
Compare
84a24bf
to
bcf30f5
Compare
node *api.Node | ||
} | ||
|
||
func PredicateMetadata(pod *api.Pod, nodeInfoMap map[string]*schedulercache.NodeInfo) interface{} { | ||
if pod == nil { | ||
// We cannot compute metadata, just return 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.
move this comment to the top of the function since now it applies to more than one return
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.
done
Just had a couple of comments. |
8807972
to
4bc410e
Compare
OK - I did some experiments and it seems that from performance point of view it's good enough for now to have this merged. |
GCE e2e build/test passed for commit 4bc410e. |
LGTM |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit 4bc410e. |
Automatic merge from submit-queue |
Ref #26144