Merge pull request #92 from packethost/BMS-698-deb10-network-fix

Debian10 network fix

Author: turegano-equinix

packetnetworking/distros/debian/bonded.py

@@ -11,7 +11,19 @@ def build(self):
         self.build_tasks()
 
     def build_tasks(self):
-        self.task_template("etc/network/interfaces", "bonded/etc_network_interfaces.j2")
+        if (
+            self.metadata.operating_system.distro == "debian"
+            and self.metadata.operating_system.version == "10"
+        ):
+            # Debian10 is failing if we don't configure the net this way
+            self.task_template(
+                "etc/network/interfaces", "bonded/etc_network_interfaces_debian10.j2"
+            )
+        else:
+            self.task_template(
+                "etc/network/interfaces", "bonded/etc_network_interfaces.j2"
+            )
+
         self.task_template("etc/modules", "bonded/etc_modules.j2", write_mode="a")
         self.tasks.update(generate_persistent_names_udev())
         return self.tasks

packetnetworking/distros/debian/conftest.py

@@ -10,7 +10,7 @@
 
 
 versions = {
-    "debian": ["10", "11", "12"],
+    "debian": ["11", "12"],
     "ubuntu": ["18.04", "20.04", "22.04"],
 }
 versions = [[distro, version] for distro in versions for version in versions[distro]]

packetnetworking/distros/debian/templates/bonded/etc_network_interfaces_debian10.j2

@@ -0,0 +1,62 @@
+auto lo
+iface lo inet loopback
+
+{% for bond in bonds | sort %}
+
+auto {{ bond }}
+iface {{ bond }} inet {% if bond == "bond0" %}static{% else %}manual{% endif %}
+
+    {% if bond == "bond0" %}
+    {% if ip4pub %}
+    address {{ ip4pub.address }}
+    netmask {{ ip4pub.netmask }}
+    gateway {{ ip4pub.gateway }}
+    {% else %}
+    address {{ ip4priv.address }}
+    netmask {{ ip4priv.netmask }}
+    gateway {{ ip4priv.gateway }}
+    {% endif %}
+    hwaddress {{ interfaces[0].mac }}
+    dns-nameservers {{ resolvers | sort | join(" ") }}
+
+    {% endif %}
+    bond-downdelay 200
+    bond-miimon 100
+    bond-mode {{ net.bonding.mode }}
+    bond-updelay 200
+    bond-xmit_hash_policy layer3+4
+    {% if net.bonding.mode == 4 %}
+    bond-lacp-rate 1
+    {% endif %}
+    bond-slaves {{ bonds[bond] | map(attribute='name') | sort | join(' ') }}
+{% if bond == "bond0" %}
+{% if ip6pub %}
+
+iface bond0 inet6 static
+    address {{ ip6pub.address }}
+    netmask {{ ip6pub.cidr }}
+    gateway {{ ip6pub.gateway }}
+{% endif %}
+{% if ip4pub %}
+
+auto bond0:0
+iface bond0:0 inet static
+    address {{ ip4priv.address }}
+    netmask {{ ip4priv.netmask }}
+    {% for subnet in private_subnets | sort %}
+    post-up route add -net {{ subnet }} gw {{ ip4priv.gateway }}
+    post-down route del -net {{ subnet }} gw {{ ip4priv.gateway }}
+    {% endfor %}
+{% endif %}
+{% endif %}
+{% endfor %}
+
+{% for iface in interfaces | sort(attribute="name") %}
+
+auto {{ iface.name }}
+iface {{ iface.name }} inet manual
+{% if iface.name != interfaces[0].name %}
+    pre-up sleep 4
+{% endif %}
+    bond-master bond0
+{% endfor %}