Skip to content

Commit

Permalink
fix: Cilium-Istio compatibility fixes (#856)
Browse files Browse the repository at this point in the history
**What problem does this PR solve?**:

Setting  `cni.exclusive: false` and `socketLB.hostNamespaceOnly: true`.

See Cilium's docs
https://docs.cilium.io/en/latest/network/servicemesh/istio/. Without
these value Cilium can interfere with Istio functionality, by always
cleaning up cni config directory on the host, thus preventing Istio Pods
from coming up, and interfere with Istio load-balancing once they do
come up.

It's safe to always set these value, because it is not Cilium's
responsibility to prevent other applications from acting like a network
plugin.

**Which issue(s) this PR fixes**:
Fixes #

**How Has This Been Tested?**:
<!--
Please describe the tests that you ran to verify your changes.
Provide output from the tests and any manual steps needed to replicate
the tests.
-->

**Special notes for your reviewer**:
<!--
Use this to provide any additional information to the reviewers.
This may include:
- Best way to review the PR.
- Where the author wants the most review attention on.
- etc.
-->

---------

Co-authored-by: Jimmi Dyson <jimmidyson@gmail.com>
  • Loading branch information
dkoshkin and jimmidyson authored Aug 14, 2024
1 parent 65df393 commit 67c6244
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
Loading

0 comments on commit 67c6244

Please sign in to comment.