Skip to content

Commit b5bc88d

Browse files
committed
Bump priority of egress ClusterIP traffic drop
We have a flow [1] to prevent leaking traffic towards a ClusterIP. However we also have a flow to prevent EIP traffic to egress before being SNATed and an additional flow to actually allow the traffic to egress in ICNI/BGP scenarios for pods on the nodes subnet [2]. The higher priority of flow [2] prevents flow [1] to be in effect. Bump priority of flow [1] since there is no case where we should leak traffic towards ClusterIPs. [1] cookie=0xdeff105, duration=492.235s, table=0, n_packets=0, n_bytes=0, priority=105,ipv6,in_port="patch-breth0_ov",ipv6_dst=fd00:10:96::/112 actions=drop [2] cookie=0xdeff105, duration=2308.615s, table=0, n_packets=4, n_bytes=376, priority=109,ipv6,in_port="patch-breth0_ov",dl_src=96:b0:34:18:12:7c,ipv6_src=fd00:10:244:1::/64 actions=ct(commit,zone=64000,exec(load:0x1->NXM_NX_CT_MARK[])),output:eth0 cookie=0xdeff105, duration=1991.854s, table=0, n_packets=0, n_bytes=0, priority=104,ipv6,in_port="patch-breth0_ov",ipv6_src=fd00:10:244::/48 actions=drop Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
1 parent 575f3c0 commit b5bc88d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

go-controller/pkg/node/gateway_shared_intf.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1676,7 +1676,7 @@ func flowsForDefaultBridge(bridge *bridgeConfiguration, extraIPs []net.IP) ([]st
16761676
// at the GR load balancer or switch load balancer. It means the correct port wasn't provided.
16771677
// nodeCIDR->serviceCIDR traffic flow is internal and it shouldn't be carried to outside the cluster
16781678
dftFlows = append(dftFlows,
1679-
fmt.Sprintf("cookie=%s, priority=105, in_port=%s, %s, %s_dst=%s,"+
1679+
fmt.Sprintf("cookie=%s, priority=115, in_port=%s, %s, %s_dst=%s,"+
16801680
"actions=drop", defaultOpenFlowCookie, netConfig.ofPortPatch, protoPrefix, protoPrefix, svcCIDR))
16811681
}
16821682
}

0 commit comments

Comments
 (0)