[pve-devel] [PATCH ifupdown2] add patch 0009-address-don-t-process_mtu-if-ovs-mtu-is-defined.patch

[Alexandre Derumier]

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 <aderumier at 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 <support at 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 <aderumier at 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 <aderumier at 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