Skip to content

helm: Remove affinity from cilium-etcd-operator#12139

Merged
joestringer merged 1 commit intomasterfrom
pr/brb/rm-etcd-affinity
Jun 18, 2020
Merged

helm: Remove affinity from cilium-etcd-operator#12139
joestringer merged 1 commit intomasterfrom
pr/brb/rm-etcd-affinity

Conversation

@brb
Copy link
Copy Markdown
Member

@brb brb commented Jun 17, 2020

The affinity section in the operator spec was introduced to avoid
scheduling an instance of the operator on the node which is not managed
by Cilium (NO_CILIUM_ON_NODE; used by the CI net-next).

We are no longer using the operator in the K8sDatapathSuite's Etcd
test case. Therefore we can remove the affinity section.

Fix #12124.

The affinity section in the operator spec was introduced to avoid
scheduling an instance of the operator on the node which is not managed
by Cilium (NO_CILIUM_ON_NODE; used by the CI net-next).

We are no longer using the operator in the K8sDatapathSuite's Etcd
test case. Therefore we can remove the affinity section.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added pending-review release-note/minor This PR changes functionality that users may find relevant to operating Cilium. area/helm Impacts helm charts and user deployment experience labels Jun 17, 2020
@brb brb requested review from a team as code owners June 17, 2020 12:13
@brb brb requested a review from a team June 17, 2020 12:13
@brb
Copy link
Copy Markdown
Member Author

brb commented Jun 17, 2020

test-me-please

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.06%) to 37.037% when pulling 5ea57c6 on pr/brb/rm-etcd-affinity into bf64aa9 on master.

@christarazi christarazi requested a review from nebril June 17, 2020 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/helm Impacts helm charts and user deployment experience release-note/minor This PR changes functionality that users may find relevant to operating Cilium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Global (anti-)affinity blocks cilium-etcd-operator from deploying on nodes with cilium

6 participants