Add node affinity and pod affinity/anti-affinity.#237
Add node affinity and pod affinity/anti-affinity.#237shashwathi wants to merge 2 commits intoknative:masterfrom shashwathi:add-affinity
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: shashwathi If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/assign @imjasonh |
imjasonh
left a comment
There was a problem hiding this comment.
I think we should try to add an e2e test for this, even if the affinity/anti-affinity is vacuously satisfied (e.g., affinity matching any node, anti-affinity matching no nodes)
That way we can at least check that specifying affinity doesn't break a build.
|
@imjasonh : Sure. I can do that
|
Fixes #116
Proposed Changes