-
Notifications
You must be signed in to change notification settings - Fork 163
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
add intelligent windows tolerations based on node taints #609
Conversation
tomkennedy513
commented
Jan 26, 2021
- we only do this if the taints across all windows nodes are homogenous
68c61f6
to
f5a5800
Compare
53415a6
to
c3714a9
Compare
should we do a provider pattern? |
sort.Slice(taintsToCompare, func(i, j int) bool { | ||
return taintsToCompare[i].Key < taintsToCompare[j].Key | ||
}) |
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.
As an optimization, maybe we can move this second sort to be inside the func taintsEqual(
, after the size check.
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.
i had that originally, but i changed it because even though its slightly less performant, i felt like it was a little weird to be sorting in the equals function, especially since i was only sorting one slice
c3714a9
to
6e68c99
Compare
- we only do this if the taints across all windows nodes are homogenous
6e68c99
to
16f65e0
Compare