summaryrefslogtreecommitdiff
path: root/smoketest/scripts
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2020-12-21 18:13:01 +0100
committerChristian Poessinger <christian@poessinger.com>2020-12-21 18:25:29 +0100
commitef629504d4b5d4bc52465b0ee2f960b63eca260e (patch)
tree34b5d4cd6d6a0ed53a7eaaa65afd4e9b19063a4d /smoketest/scripts
parentf8db20706527ab04f3585ab28bc2bed030c7def7 (diff)
downloadvyos-1x-ef629504d4b5d4bc52465b0ee2f960b63eca260e.tar.gz
vyos-1x-ef629504d4b5d4bc52465b0ee2f960b63eca260e.zip
smoketest: mirror: drop testcase
The mirror smoketest does more harm then good. It creates interfaces which are not cleaned up afterwards resulting and strange behavior on live systems. The entrie interface mirror smoketest must be redone.
Diffstat (limited to 'smoketest/scripts')
-rw-r--r--smoketest/scripts/cli/base_interfaces_test.py50
1 files changed, 0 insertions, 50 deletions
diff --git a/smoketest/scripts/cli/base_interfaces_test.py b/smoketest/scripts/cli/base_interfaces_test.py
index fd37cb2a1..0ae55a22a 100644
--- a/smoketest/scripts/cli/base_interfaces_test.py
+++ b/smoketest/scripts/cli/base_interfaces_test.py
@@ -32,23 +32,6 @@ from vyos.util import process_named_running
from vyos.validate import is_intf_addr_assigned
from vyos.validate import is_ipv6_link_local
-def read_mirror_rule(interfaces):
- Success = 0
- for interface in interfaces:
- get_tc_cmd = 'tc -j qdisc'
- tmp = cmd(get_tc_cmd, shell=True)
- data = json.loads(tmp)
- for rule in data:
- dev = rule['dev']
- handle = rule['handle']
- kind = rule['kind']
- if dev == interface and handle == "ffff:" and kind == "ingress":
- Success+=1
- elif dev == interface and handle == "1:" and kind == "prio":
- Success+=1
- return Success
-
-
dhcp6c_config_file = '/run/dhcp6c/dhcp6c.{}.conf'
def get_dhcp6c_config_value(interface, key):
tmp = read_file(dhcp6c_config_file.format(interface))
@@ -100,39 +83,6 @@ class BasicInterfaceTest:
self.session.commit()
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
- for interface in self._interfaces:
- self.session.set(self._base_path + [interface, 'mirror', 'ingress', 'bond1'])
- self.session.set(self._base_path + [interface, 'mirror', 'egress', 'bond1'])
- i+=1
- self.session.commit()
- # Parse configuration
- Success = read_mirror_rule(self._interfaces)
- if Success == i*2:
- self.assertTrue(True)
- else:
- self.assertTrue(False)
- i=0
- self.session.delete(['interfaces','dummy'])
- self.session.delete(['interfaces','bonding'])
-
-
def test_add_description(self):
"""
Check if description can be added to interface