controller: fix DHCP MTU when the default network mode is underlay (#…

…2941)
kubeovn · Jun 14, 2023 · ea5f81a · ea5f81a
1 parent 6d883dc
commit ea5f81a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/pkg/controller/subnet.go b/pkg/controller/subnet.go
@@ -686,24 +686,24 @@ func (c *Controller) handleAddOrUpdateSubnet(key string) error {
 		}
 	}
 
-	var dhcpOptionsUUIDs *ovs.DHCPOptionsUUIDs
-	var mtu int
-	if subnet.Spec.Vlan != "" {
-		mtu = 1500
-	} else {
+	mtu := util.DefaultMTU
+	if subnet.Spec.Vlan == "" {
 		switch c.config.NetworkType {
-		case util.NetworkTypeVxlan:
-			mtu = 1500 - util.VxlanHeaderLength
+		case util.NetworkTypeVlan:
+			// default to geneve
+			fallthrough
 		case util.NetworkTypeGeneve:
-			mtu = 1500 - util.GeneveHeaderLength
+			mtu -= util.GeneveHeaderLength
+		case util.NetworkTypeVxlan:
+			mtu -= util.VxlanHeaderLength
 		case util.NetworkTypeStt:
-			mtu = 1500 - util.SttHeaderLength
+			mtu -= util.SttHeaderLength
 		default:
 			return fmt.Errorf("invalid network type: %s", c.config.NetworkType)
 		}
 	}
 
-	dhcpOptionsUUIDs, err = c.ovnLegacyClient.UpdateDHCPOptions(subnet.Name, subnet.Spec.CIDRBlock, subnet.Spec.Gateway, subnet.Spec.DHCPv4Options, subnet.Spec.DHCPv6Options, subnet.Spec.EnableDHCP, mtu)
+	dhcpOptionsUUIDs, err := c.ovnLegacyClient.UpdateDHCPOptions(subnet.Name, subnet.Spec.CIDRBlock, subnet.Spec.Gateway, subnet.Spec.DHCPv4Options, subnet.Spec.DHCPv6Options, subnet.Spec.EnableDHCP, mtu)
 	if err != nil {
 		klog.Errorf("failed to update dhcp options for switch %s, %v", subnet.Name, err)
 		return err

diff --git a/pkg/util/const.go b/pkg/util/const.go
@@ -124,6 +124,8 @@ const (
 	SubnetAllowPriority = "1001"
 	DefaultDropPriority = "1000"
 
+	DefaultMTU = 1500
+
 	GeneveHeaderLength = 100
 	VxlanHeaderLength  = 50
 	SttHeaderLength    = 72