summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRyan Harper <ryan.harper@canonical.com>2018-03-16 14:00:15 -0400
committerScott Moser <smoser@brickies.net>2018-03-16 14:00:15 -0400
commitb7b7331b9c308d8e1cb0b3dfd4398e6e7cb1b60f (patch)
tree876fd67b90b67f95f9dfe8682d53c846d5874700
parent97012fbb5207ddf1d2dcbb1c5eae710c47ab8ec0 (diff)
downloadvyos-cloud-init-b7b7331b9c308d8e1cb0b3dfd4398e6e7cb1b60f.tar.gz
vyos-cloud-init-b7b7331b9c308d8e1cb0b3dfd4398e6e7cb1b60f.zip
netplan: render bridge port-priority values
Update netplan renderer to write out bridge port-priority values now that netplan supports the feature. LP: #1735821
-rw-r--r--cloudinit/net/netplan.py10
-rw-r--r--cloudinit/net/network_state.py2
-rw-r--r--tests/unittests/test_net.py3
3 files changed, 9 insertions, 6 deletions
diff --git a/cloudinit/net/netplan.py b/cloudinit/net/netplan.py
index d3788af8..6bee3d37 100644
--- a/cloudinit/net/netplan.py
+++ b/cloudinit/net/netplan.py
@@ -311,12 +311,12 @@ class Renderer(renderer.Renderer):
if newname is None:
continue
br_config.update({newname: value})
- if newname == 'path-cost':
- # <interface> <cost> -> <interface>: int(<cost>)
+ if newname in ['path-cost', 'port-priority']:
+ # <interface> <value> -> <interface>: int(<value>)
newvalue = {}
- for costval in value:
- (port, cost) = costval.split()
- newvalue[port] = int(cost)
+ for val in value:
+ (port, portval) = val.split()
+ newvalue[port] = int(portval)
br_config.update({newname: newvalue})
if len(br_config) > 0:
diff --git a/cloudinit/net/network_state.py b/cloudinit/net/network_state.py
index fe667d88..1dd7ded7 100644
--- a/cloudinit/net/network_state.py
+++ b/cloudinit/net/network_state.py
@@ -47,7 +47,7 @@ NET_CONFIG_TO_V2 = {
'bridge_maxage': 'max-age',
'bridge_maxwait': None,
'bridge_pathcost': 'path-cost',
- 'bridge_portprio': None,
+ 'bridge_portprio': 'port-priority',
'bridge_stp': 'stp',
'bridge_waitport': None}}
diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py
index ac33e8ef..9cf11f2e 100644
--- a/tests/unittests/test_net.py
+++ b/tests/unittests/test_net.py
@@ -758,6 +758,9 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
path-cost:
eth3: 50
eth4: 75
+ port-priority:
+ eth3: 28
+ eth4: 14
priority: 22
stp: false
routes: