diff options
author | Christian Poessinger <christian@poessinger.com> | 2022-04-06 09:40:19 +0200 |
---|---|---|
committer | Christian Poessinger <christian@poessinger.com> | 2022-04-06 10:08:34 +0200 |
commit | af2563589b79539fd771c167aee0ab531a8f0fba (patch) | |
tree | af45a02c0ec95af4d3b52e193a25decbf18500ee /smoketest | |
parent | adda3d86080bb3b8fe26424c7d829e40eb9e2b23 (diff) | |
download | vyos-1x-af2563589b79539fd771c167aee0ab531a8f0fba.tar.gz vyos-1x-af2563589b79539fd771c167aee0ab531a8f0fba.zip |
smoketest: vrf: T4331: IPv6 link-local addresses not configured for interface in VRF
(cherry picked from commit 5b57c536b5f599deea2106aad7aea95c465bc0c0)
Diffstat (limited to 'smoketest')
-rwxr-xr-x | smoketest/scripts/cli/test_vrf.py | 49 |
1 files changed, 45 insertions, 4 deletions
diff --git a/smoketest/scripts/cli/test_vrf.py b/smoketest/scripts/cli/test_vrf.py index 0f006ca3c..5a44fdc66 100755 --- a/smoketest/scripts/cli/test_vrf.py +++ b/smoketest/scripts/cli/test_vrf.py @@ -29,6 +29,7 @@ from vyos.ifconfig import Section from vyos.template import is_ipv6 from vyos.util import cmd from vyos.util import read_file +from vyos.util import get_interface_config from vyos.validate import is_intf_addr_assigned base_path = ['vrf'] @@ -176,11 +177,11 @@ class VRFTest(VyOSUnitTestSHIM.TestCase): # commit changes self.cli_commit() - # Verify & cleanup + # Verify VRF assignmant for interface in self._interfaces: - # os.readlink resolves to: '../../../../../virtual/net/foovrf' - tmp = os.readlink(f'/sys/class/net/{interface}/master').split('/')[-1] - self.assertEqual(tmp, vrf) + tmp = get_interface_config(interface) + self.assertEqual(vrf, tmp['master']) + # cleanup section = Section.section(interface) self.cli_delete(['interfaces', section, interface, 'vrf']) @@ -256,5 +257,45 @@ class VRFTest(VyOSUnitTestSHIM.TestCase): table = str(int(table) + 1) + def test_vrf_link_local_ip_addresses(self): + # Testcase for issue T4331 + table = '100' + vrf = 'orange' + interface = 'dum9998' + addresses = ['192.0.2.1/26', '2001:db8:9998::1/64', 'fe80::1/64'] + + for address in addresses: + self.cli_set(['interfaces', 'dummy', interface, 'address', address]) + + # Create dummy interfaces + self.cli_commit() + + # ... and verify IP addresses got assigned + for address in addresses: + self.assertTrue(is_intf_addr_assigned(interface, address)) + + # Move interface to VRF + self.cli_set(base_path + ['name', vrf, 'table', table]) + self.cli_set(['interfaces', 'dummy', interface, 'vrf', vrf]) + + # Apply VRF config + self.cli_commit() + # Ensure VRF got created + self.assertIn(vrf, interfaces()) + # ... and IP addresses are still assigned + for address in addresses: + self.assertTrue(is_intf_addr_assigned(interface, address)) + # Verify VRF table ID + tmp = get_interface_config(vrf) + self.assertEqual(int(table), tmp['linkinfo']['info_data']['table']) + + # Verify interface is assigned to VRF + tmp = get_interface_config(interface) + self.assertEqual(vrf, tmp['master']) + + # Delete Interface + self.cli_delete(['interfaces', 'dummy', interface]) + self.cli_commit() + if __name__ == '__main__': unittest.main(verbosity=2) |