Skip to content

Commit

Permalink
cni: Revert "cni: Use correct route MTU for various cloud cidrs"
Browse files Browse the repository at this point in the history
The PR cilium#32244, that was merged with commit 29a340e, was intended to fix
IP fragmentation with WireGuard deployments, causing poor network
throughput and increased network latency. Unfortunately, after this PR
was merged, users began reporting issues with Cilium modifying the MTU
of the default interface of the node. This commit reverts the blamed
commit in an attempt to fix said issues.

The surfaced side-effect is tracked in issue cilium#33303.

Fixes: cilium#33258

Signed-off-by: Ryan Drew <ryan.drew@isovalent.com>
  • Loading branch information
learnitall authored and aanm committed Jun 24, 2024
1 parent 50f29b3 commit a31dcbd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion plugins/cilium-cni/cmd/interface.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ func interfaceAdd(ipConfig *current.IPConfig, ipam *models.IPAMAddressResponse,

if err := routingInfo.Configure(
ipConfig.Address.IP,
int(conf.RouteMTU),
int(conf.DeviceMTU),
conf.EgressMultiHomeIPRuleCompat,
false,
); err != nil {
Expand Down

0 comments on commit a31dcbd

Please sign in to comment.