cloud-init-dev team mailing list archive

[Ryan Harper <ryan.harper@xxxxxxxxxxxxx>]

Shouldn't we also add  lines 9 and 10, to the the 'bridge' section of netplan rendering?


% git diff
diff --git a/cloudinit/net/netplan.py b/cloudinit/net/netplan.py
index 4014363..35b16c3 100644
--- a/cloudinit/net/netplan.py
+++ b/cloudinit/net/netplan.py
@@ -291,6 +291,9 @@ class Renderer(renderer.Renderer):
 
                 if len(bond_config) > 0:
                     bond.update({'parameters': bond_config})
+                macaddr = ifcfg.get('mac_address', None)
+                if macaddr is not None:
+                    bond['macaddress'] = macaddr.lower()
                 slave_interfaces = ifcfg.get('bond-slaves')
                 if slave_interfaces == 'none':
                     _extract_bond_slaves_by_name(interfaces, bond, ifname)
@@ -327,6 +330,9 @@ class Renderer(renderer.Renderer):
 
                 if len(br_config) > 0:
                     bridge.update({'parameters': br_config})
+                macaddr = ifcfg.get('mac_address', None)
+                if macaddr is not None:
+                    bridge['macaddress'] = macaddr.lower()
                 _extract_addresses(ifcfg, bridge, ifname)
                 bridges.update({ifname: bridge})

-- 
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/352033
Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:bug/1784699-netplan-macaddress-on-bond into cloud-init:master.