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
install/kubernetes: do not schedule cilium-operator pods in same node #12771
Conversation
test-me-please |
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.
LGTM, simple enough change. Small nits that can be addressed after CI runs, and no need for a re-review.
Can we change the release-note to say "avoid" instead of "do not"?
install/kubernetes/cilium/charts/operator/templates/deployment.yaml
Outdated
Show resolved
Hide resolved
Coverage decreased (-0.02%) to 36.839% when pulling 4189fc4f1180e4c008e79191f3d2803ae9e64397 on aanm:pr/fix-cilium-operator-ha into 9606ff5 on cilium:master. |
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.
LGTM. Chris' comment does clarify a bit.
We'll wait for CI results then we can fix up the cosmetic nit & merge (assuming CI is happy). |
Since Cilium Operator is running in host network, 2 or more pods can't run in the same node at the same time or they will clash the open ports they use for liveness and / or readiness health check. Fixes: 930bde7 ("install: update helm templates to add HA capabilities for operator") Signed-off-by: André Martins <andre@cilium.io>
4189fc4
to
dedfd24
Compare
CI was green. I've pushed the requested changes which doesn't require another CI run. |
Since Cilium Operator is running in host network, 2 or more pods can't
run in the same node at the same time or they will clash the open
ports they use for liveness and / or readiness health check.
Fixes: 930bde7 ("install: update helm templates to add HA capabilities for operator")
Signed-off-by: André Martins andre@cilium.io