From: Ondrej Lichtner <olichtne(a)redhat.com>
Removing a bit of forgotten code - when I split the down method into
down + address_cleanup I forgot to remove the "ip addr del" commands for
internal ports from the down method resulting in calling the same
command twice. The second call would fail and throw an exception since
the address would already be removed.
Fixes: 82f606fb ("{Net, Nm}ConfigDevice: split up() into up+address_setup")
Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com>
---
lnst/Slave/NetConfigDevice.py | 3 ---
1 file changed, 3 deletions(-)
diff --git a/lnst/Slave/NetConfigDevice.py b/lnst/Slave/NetConfigDevice.py
index 9681ac5..a668e32 100644
--- a/lnst/Slave/NetConfigDevice.py
+++ b/lnst/Slave/NetConfigDevice.py
@@ -464,9 +464,6 @@ class NetConfigDeviceOvsBridge(NetConfigDeviceGeneric):
int_ports = self._dev_config["ovs_conf"]["internals"]
br_name = self._dev_config["name"]
for iport in int_ports:
- if "addresses" in iport:
- for address in iport["addresses"]:
- exec_cmd("ip addr del %s dev %s" % (address,
iport["name"]))
exec_cmd("ip link set %s down" % iport["name"])
super(NetConfigDeviceOvsBridge, self).down()
--
2.10.1