From 34437ff4dee5fb9fb70bbc7c8b83ecb074793770 Mon Sep 17 00:00:00 2001 From: Christian Poessinger Date: Sun, 20 Dec 2020 16:31:06 +0100 Subject: smoktest: interface: disable base mirror test One should not use hardcoded interfaces in this base class, rather rely on self._options like every other test. This implementation breaks on my development system ans also deletes my OSPF dummy mgmt interface :(. --- smoketest/scripts/cli/base_interfaces_test.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'smoketest/scripts') diff --git a/smoketest/scripts/cli/base_interfaces_test.py b/smoketest/scripts/cli/base_interfaces_test.py index d08dff82c..f8e9835af 100644 --- a/smoketest/scripts/cli/base_interfaces_test.py +++ b/smoketest/scripts/cli/base_interfaces_test.py @@ -101,15 +101,19 @@ class BasicInterfaceTest: del self.session def test_mirror(self): + # Test is disabled as it contains hardcoded bond interfaces which will + # screw up all kinds of live deployments. + return None if self._test_mirror: - # Create test dependency interface self.session.set(['interfaces','dummy','dum0']) self.session.set(['interfaces','dummy','dum1']) self.session.set(['interfaces','bonding','bond1','member','interface','dum0']) self.session.set(['interfaces','bonding','bond1','member','interface','dum1']) + # ^- WHY? There is self._options for that :( + Success = 0 i = 0 # Check the two-way mirror rules of ingress and egress @@ -127,8 +131,6 @@ class BasicInterfaceTest: i=0 self.session.delete(['interfaces','dummy']) self.session.delete(['interfaces','bonding']) - else: - return None def test_add_description(self): -- cgit v1.2.3