Skip to content

Commit

Permalink
Reset the rp_filter value for "all" as well (on the container side).
Browse files Browse the repository at this point in the history
That setting is applied as an override. The effective value on a given interface
is the max of the interface's value and the "all" value. Makes no sense, but
that's the way it works.
  • Loading branch information
jiceatscion committed Mar 21, 2024
1 parent c9d4e87 commit 6a23f19
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion acceptance/router_benchmark/test.py
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,6 @@ def create_interface(self, req: IntfReq, ns: str):
"dev", hostIntf, "scope", "link")

sudo("sysctl", "-qw", f"net.ipv6.conf.{hostIntf}.disable_ipv6=1")
sudo("sysctl", "-qw", f"net.ipv4.conf.{hostIntf}.rp_filter=0")
sudo("ethtool", "-K", brIntf, "rx", "off", "tx", "off")
sudo("ip", "link", "set", brIntf, "mtu", "8000")
sudo("ip", "link", "set", brIntf, "address", mac)
Expand All @@ -338,6 +337,8 @@ def create_interface(self, req: IntfReq, ns: str):
sudo("ip", "link", "set", brIntf, "netns", ns)
sudo("ip", "netns", "exec", ns,
"sysctl", "-qw", f"net.ipv6.conf.{brIntf}.disable_ipv6=1")
sudo("ip", "netns", "exec", ns,
"sysctl", "-qw", f"net.ipv4.conf.all.rp_filter=0")
sudo("ip", "netns", "exec", ns,
"sysctl", "-qw", f"net.ipv4.conf.{brIntf}.rp_filter=0")

Expand Down

0 comments on commit 6a23f19

Please sign in to comment.