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.
_______________________________________________
Mailing list: https://launchpad.net/~cloud-init-dev
Post to : cloud-init-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help : https://help.launchpad.net/ListHelp
