https://github.com/CumulusNetworks/ifupdown2/pull/166
Openvswitch already manage mtu if ovs-mtu is defined. (Ovs manage mtu in userland, and sync mtu for some interfaces in kernel). If mtu is changed by address module, before the ovs userland mtu, this give packets drop. Signed-off-by: Alexandre Derumier <aderum...@odiso.com> --- debian/changelog | 6 ++++ ...-t-process_mtu-if-ovs-mtu-is-defined.patch | 32 +++++++++++++++++++ debian/patches/series | 1 + 3 files changed, 39 insertions(+) create mode 100644 debian/patches/pve/0009-address-don-t-process_mtu-if-ovs-mtu-is-defined.patch diff --git a/debian/changelog b/debian/changelog index c5b2f2a..29f0ab8 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +ifupdown2 (3.0.0-1+pve2) pve; urgency=medium + + * ovs-mtu fix + + -- Proxmox Support Team <supp...@proxmox.com> Thu, 04 Jun 2020 13:00:01 +0200 + ifupdown2 (3.0.0-1+pve1) pve; urgency=medium * update to new 3.0.0 upstream release diff --git a/debian/patches/pve/0009-address-don-t-process_mtu-if-ovs-mtu-is-defined.patch b/debian/patches/pve/0009-address-don-t-process_mtu-if-ovs-mtu-is-defined.patch new file mode 100644 index 0000000..d12a6ed --- /dev/null +++ b/debian/patches/pve/0009-address-don-t-process_mtu-if-ovs-mtu-is-defined.patch @@ -0,0 +1,32 @@ +From 54dbee161aa2fd281046386c10123222b790ebfc Mon Sep 17 00:00:00 2001 +From: Alexandre Derumier <aderum...@odiso.com> +Date: Wed, 3 Jun 2020 13:06:14 +0200 +Subject: [PATCH] address: don't process_mtu if ovs-mtu is defined + +Openvswitch already manage mtu if ovs-mtu is defined. +(Ovs manage mtu in userland, and sync mtu for some interfaces in kernel). + +Signed-off-by: Alexandre Derumier <aderum...@odiso.com> +--- + ifupdown2/addons/address.py | 5 +++++ + 1 file changed, 5 insertions(+) + +diff --git a/ifupdown2/addons/address.py b/ifupdown2/addons/address.py +index 6578ebb..9a20c9e 100644 +--- a/ifupdown2/addons/address.py ++++ b/ifupdown2/addons/address.py +@@ -881,6 +881,11 @@ class address(Addon, moduleBase): + + def process_mtu(self, ifaceobj, ifaceobj_getfunc): + mtu_str = ifaceobj.get_attr_value_first('mtu') ++ ++ ovs_mtu = ifaceobj.get_attr_value_first('ovs-mtu') ++ if ovs_mtu: ++ return ++ + mtu_from_policy = False + + if not mtu_str: +-- +2.20.1 + diff --git a/debian/patches/series b/debian/patches/series index a83e94e..b454408 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -6,4 +6,5 @@ pve/0005-ifreload-down-up-vxlan-interfaces-when-ifreload_down.patch pve/0006-config-tuning.patch pve/0007-networking.service-fix-dependencies-and-ordering.patch pve/0008-openvswitch-ovs-ports-condone-regex-exclude-tap-veth.patch +pve/0009-address-don-t-process_mtu-if-ovs-mtu-is-defined.patch pve/0010-postinst-rm-update-network-config-compatibility.patch -- 2.20.1 _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel