summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2021-01-03 11:09:04 +0100
committerChristian Poessinger <christian@poessinger.com>2021-01-03 11:09:18 +0100
commitea87ba4b0be72df958166bdf8d9d5116f60bc68e (patch)
treef9e99a2c5eed9285595076df21da4dea99378a0a
parentbc599ab4610f8bfd5e19a9c4cfe6bb50b4ceb0b0 (diff)
downloadvyos-1x-ea87ba4b0be72df958166bdf8d9d5116f60bc68e.tar.gz
vyos-1x-ea87ba4b0be72df958166bdf8d9d5116f60bc68e.zip
Partial Revert "interfaces: T3114: Fix VLAN-aware bridge setting failure"
This reverts parts of commit c4048b2047f693436b38196632cddf65beb67a86 because when tear-downing config session entire interface paths are removed with non test related interface (all dummy interfaces) this breaks e.g. remote connections when connected to one of the dummy interface IP addresses for management.
-rwxr-xr-xsmoketest/scripts/cli/test_interfaces_bridge.py19
1 files changed, 0 insertions, 19 deletions
diff --git a/smoketest/scripts/cli/test_interfaces_bridge.py b/smoketest/scripts/cli/test_interfaces_bridge.py
index 04a117edf..21800e508 100755
--- a/smoketest/scripts/cli/test_interfaces_bridge.py
+++ b/smoketest/scripts/cli/test_interfaces_bridge.py
@@ -46,20 +46,10 @@ class BridgeInterfaceTest(BasicInterfaceTest.BaseTest):
if not '.' in tmp:
self._members.append(tmp)
- 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'])
- self._members.append('bond1')
-
self._options['br0'] = []
for member in self._members:
self._options['br0'].append(f'member interface {member}')
- def tearDown(self):
- self.session.delete(['interfaces','bonding'])
- self.session.delete(['interfaces','dummy'])
- super().tearDown()
def test_add_remove_bridge_member(self):
# Add member interfaces to bridge and set STP cost/priority
@@ -123,14 +113,10 @@ class BridgeInterfaceTest(BasicInterfaceTest.BaseTest):
self.assertEqual(int(flags), 1)
# Execute the program to obtain status information
-
json_data = cmd('bridge -j vlan show', shell=True)
-
vlan_filter_status = None
-
vlan_filter_status = json.loads(json_data)
-
if vlan_filter_status is not None:
for interface_status in vlan_filter_status:
ifname = interface_status['ifname']
@@ -164,11 +150,7 @@ class BridgeInterfaceTest(BasicInterfaceTest.BaseTest):
else:
self.assertTrue(False)
-
-
-
# check member interfaces are added on the bridge
-
for interface in self._interfaces:
bridge_members = []
for tmp in glob(f'/sys/class/net/{interface}/lower_*'):
@@ -205,7 +187,6 @@ class BridgeInterfaceTest(BasicInterfaceTest.BaseTest):
self.session.delete(['interfaces', 'ethernet', 'eth0', 'vif', '400'])
self.session.commit()
-
if __name__ == '__main__':
unittest.main(verbosity=2)