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
Fixes to use empty labelSelector #557
Fixes to use empty labelSelector #557
Conversation
d0cab16
to
6c0559b
Compare
}, | ||
}, | ||
// DefaultNodeSelector is the NodeAffinity to be used when labelSelector is nil | ||
DefaultNodeSelector = &corev1.NodeAffinity{ |
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.
If it is DefaultNodeAffinity
, why name it DefaultNodeSelector
?
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.
Ah! Will fix that
6c0d3ed
to
27211e6
Compare
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.
Overall looks good. For the TestGetPlacement()
test, add another case #5 to test that an empty but non-nil LabelSelector sets no NodeAffinity in the resulting Placement.
Signed-off-by: Anubhav Deep <adeep@redhat.com>
Signed-off-by: Anubhav Deep <adeep@redhat.com>
Signed-off-by: Anubhav Deep <adeep@redhat.com>
27211e6
to
578142f
Compare
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.
Looks good!
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: anubhav-here, jarrpa 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 |
/cherrypick release-4.5 |
@raghavendra-talur: new pull request created: #575 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Usage doc : https://docs.google.com/document/d/1k6CMTSy297Ta8P9lZW5FPQ7y4se65rY_9WtMqyaBMag/edit?usp=sharing
Closes #553