Skip to content
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

Cilium L2 Announcements compatibility #3093

Closed
vixns opened this issue Apr 4, 2024 · 4 comments
Closed

Cilium L2 Announcements compatibility #3093

vixns opened this issue Apr 4, 2024 · 4 comments

Comments

@vixns
Copy link
Contributor

vixns commented Apr 4, 2024

We are using EnvoyGateway on cilium load balancer with L2 Announcements.

Cilium uses leases to assign the IP to a node, as L2 LB is more a fail over IP than a real load balancer.
When there is no envoy(gateway) pod on this node, the gateway is not reachable on the external/LB IP.

Our current current workaround is setting envoyDeployment.replicas to the dataplane's node count.

One way to address this would be to run envoy as DaemonSet, simple and efficient, but we may not want to run EnvoyGateway on all/some nodes.

A better way would be to automatically run an Envoy pod on all nodes matching the corresponding CiliumL2AnnouncementPolicy CR's selector.

https://docs.cilium.io/en/stable/network/l2-announcements/

@vixns vixns added the triage label Apr 4, 2024
@shawnh2
Copy link
Contributor

shawnh2 commented Apr 5, 2024

this is quite similar to #2969

Copy link

github-actions bot commented May 5, 2024

This issue has been automatically marked as stale because it has not had activity in the last 30 days.

@github-actions github-actions bot added the stale label May 5, 2024
@zvlb
Copy link
Contributor

zvlb commented May 6, 2024

#3092 - run envoy-gatewat like daemonset already merged

@zirain
Copy link
Contributor

zirain commented May 6, 2024

#3092 should help this, please reopen if need more work.

@zirain zirain closed this as completed May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants