-
Notifications
You must be signed in to change notification settings - Fork 39k
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 requirements based scheduling. #1946
Conversation
1e1127e
to
2f3e5a2
Compare
@@ -367,6 +367,8 @@ type Pod struct { | |||
Labels map[string]string `json:"labels,omitempty" yaml:"labels,omitempty"` | |||
DesiredState PodState `json:"desiredState,omitempty" yaml:"desiredState,omitempty"` | |||
CurrentState PodState `json:"currentState,omitempty" yaml:"currentState,omitempty"` | |||
// NodeSelector is a selector which must be true for the pod to fit on a node | |||
NodeSelector map[string]string `json:"nodeSelector,omitempty" yaml:"nodeSelector,omitempty"` |
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.
What are you going to use this selector to accomplish? @erictune and I were talking about something similar based on the presence of a trust relationship in the auth stuff.
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.
Look at the changes to predicates. This is so that we can do constraint/requirements based scheduling.
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.
See #367.
Is there a reason why you aren't adding this to the default predicate list? |
2f3e5a2
to
58a2e37
Compare
I'd rather submit the predicate first, and then activate it in a different PR. That way if it breaks, it's easier to just rollback the activation, rather than the entire PR. Other comment addressed. Please re-check. Thanks! |
LGTM |
58a2e37
to
fb40ae2
Compare
fb40ae2
to
5d4d607
Compare
Add requirements based scheduling.
OCPBUGS-32299: Bump to k8s 1.28.9
Adds scheduling constraints. See #367 for more details.