-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Avoid full sort when selecting host with highest priority. #16647
Conversation
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
1 similar comment
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
Labelling this PR as size/M |
@kubernetes/goog-control-plane @kubernetes/rh-cluster-infra |
LGTM |
@k8s-bot ok to test
|
GCE e2e build/test failed for commit c64048d. |
@k8s-bot e2e test this please |
GCE e2e test build/test passed for commit c64048d. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e test build/test passed for commit c64048d. |
Automatic merge from submit-queue |
Auto commit by PR queue bot
I was reading through the scheduling code and figured I'd offer a little optimization when selecting the highest priority host. I wouldn't argue that this is strictly needed, but I think it's a little better and not overly complicated. There aren't added tests because the output is the same and already tested.