-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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 and minor cleanups to pod (anti)affinity predicate #45098
Fixes and minor cleanups to pod (anti)affinity predicate #45098
Conversation
/lgtm Thanks, @wojtek-t! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bsalamat, wojtek-t
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@k8s-bot unit test this |
@bsalamat - thanks; though to be clear, this is not fixing the main issue we were discussing, it's still not debugged |
Yes, I am aware that this is not addressing that issue. |
LGTM |
LGTM. |
btw, it seems the unit test did not check this issue. Should we add one for it? I can help to create PR to update unit test as a follow up. |
Sure, if you can create a test case for it that would be great. |
@k8s-bot bazel test this |
Automatic merge from submit-queue |
#45152 was created for unit test of this fix. |
Automatic merge from submit-queue (batch tested with PRs 45100, 45152, 42513, 44796, 45222) Added InterPodAffinity unit test case with Namespace. **What this PR does / why we need it**: Added InterPodAffinity unit test case with Namespace: unit test case for #45098 **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # N/A **Release note**: ```release-note-none ```
No description provided.