-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed
Labels
area/daemonImpacts operation of the Cilium daemon.Impacts operation of the Cilium daemon.help-wantedPlease volunteer for this by adding yourself as an assignee!Please volunteer for this by adding yourself as an assignee!kind/enhancementThis would improve or streamline existing functionality.This would improve or streamline existing functionality.
Description
As described in [1], some of Cilium SNAT/MASQUERADE iptables rules could benefit from --random-fully
(TL;DR two concurrent requests sent from two different netns to the same outside destination might race for the same tuple resulting in one request being dropped by netfilter).
Before adding the param to the rules, we shoud:
- Check whether the 4.9 kernel (min required vsn by Cilium) supports the param (i.e. whether [2] is included in 4.9). If not, then probing should be implemented.
- Identify which rules should be patched (i.e. those which might handle traffic from non-host netns).
[1]: https://tech.xing.com/a-reason-for-unexplained-connection-timeouts-on-kubernetes-docker-abd041cf7e02
[2]: http://patchwork.ozlabs.org/patch/304306/
Metadata
Metadata
Assignees
Labels
area/daemonImpacts operation of the Cilium daemon.Impacts operation of the Cilium daemon.help-wantedPlease volunteer for this by adding yourself as an assignee!Please volunteer for this by adding yourself as an assignee!kind/enhancementThis would improve or streamline existing functionality.This would improve or streamline existing functionality.