Edward Haas has uploaded a new change for review.

Change subject: net func tests: ip config drop on bond slaves
......................................................................

net func tests: ip config drop on bond slaves

Moving an interface to a bond should flush/drop
its ip configuration.

Existing test changed to reflect this logic.
[test_drop_initial_bond_slaves_ip_config]

Change-Id: I17e848d86ab5423ac885f1d3db81d1e72c7a2dbe
Signed-off-by: Edward Haas <edwa...@redhat.com>
---
M tests/functional/networkTests.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/39/49539/1

diff --git a/tests/functional/networkTests.py b/tests/functional/networkTests.py
index be27dc2..8a44802 100644
--- a/tests/functional/networkTests.py
+++ b/tests/functional/networkTests.py
@@ -2680,7 +2680,7 @@
 
     @cleanupNet
     @ValidateRunningAsRoot
-    def test_keep_initial_bond_slaves_ip_config(self):
+    def test_drop_initial_bond_slaves_ip_config(self):
         with dummyIf(2) as nics:
             nic_1, nic_2 = nics
             addrAdd(nic_1, IP_ADDRESS, IP_CIDR)
@@ -2692,8 +2692,8 @@
 
                 ipv4addrs = self.vdsm_net.netinfo.nics[nic_1]['ipv4addrs']
                 ipv6addrs = self.vdsm_net.netinfo.nics[nic_1]['ipv6addrs']
-                self.assertIn(IP_ADDRESS_AND_CIDR, ipv4addrs)
-                self.assertIn(IPv6_ADDRESS_AND_CIDR, ipv6addrs)
+                self.assertEqual([], ipv4addrs)
+                self.assertEqual([], ipv6addrs)
 
                 status, msg = self.vdsm_net.setupNetworks(
                     {}, {BONDING_NAME: {'remove': True}}, NOCHK)


-- 
To view, visit https://gerrit.ovirt.org/49539
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I17e848d86ab5423ac885f1d3db81d1e72c7a2dbe
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to