Skip to content

Commit 8777b58

Browse files
authored
Merge pull request #5227 from crnithya/pmtud_offload
gateway_mtu should be set on router ports only if node supports it
2 parents fa233dc + a13d977 commit 8777b58

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

go-controller/pkg/ovn/base_network_controller.go

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -386,13 +386,19 @@ func (bnc *BaseNetworkController) syncNodeClusterRouterPort(node *corev1.Node, h
386386
gwIfAddr := util.GetNodeGatewayIfAddr(hostSubnet)
387387
lrpNetworks = append(lrpNetworks, gwIfAddr.String())
388388
}
389+
390+
var lrpOptions map[string]string
391+
enableGatewayMTU := util.ParseNodeGatewayMTUSupport(node)
392+
if enableGatewayMTU {
393+
lrpOptions = map[string]string{
394+
"gateway_mtu": strconv.Itoa(config.Default.MTU),
395+
}
396+
}
389397
logicalRouterPort := nbdb.LogicalRouterPort{
390398
Name: lrpName,
391399
MAC: nodeLRPMAC.String(),
392400
Networks: lrpNetworks,
393-
Options: map[string]string{
394-
"gateway_mtu": strconv.Itoa(config.Default.MTU),
395-
},
401+
Options: lrpOptions,
396402
}
397403
logicalRouter := nbdb.LogicalRouter{Name: logicalRouterName}
398404
gatewayChassis := nbdb.GatewayChassis{

0 commit comments

Comments
 (0)