Bond: T6303: fixed smoke-test and space

vyos · May 6, 2024 · 5b0ecee · 5b0ecee
1 parent 0b38e99
commit 5b0ecee
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/python/vyos/ifconfig/bond.py b/python/vyos/ifconfig/bond.py
@@ -55,12 +55,10 @@ class BondIf(Interface):
             'validate': lambda v: assert_list(v, ['slow', 'fast']),
             'location': '/sys/class/net/{ifname}/bonding/lacp_rate',
         },
-
         'bond_system_mac': {
             'validate': assert_mac,
             'location': '/sys/class/net/{ifname}/bonding/ad_actor_system',
         },
-
         'bond_miimon': {
             'validate': assert_positive,
             'location': '/sys/class/net/{ifname}/bonding/miimon'

diff --git a/smoketest/scripts/cli/test_interfaces_bonding.py b/smoketest/scripts/cli/test_interfaces_bonding.py
@@ -254,9 +254,8 @@ def test_bonding_system_mac(self):
 
         # verify config
         for interface in self._interfaces:
-            slaves = read_file(f'/sys/class/net/{interface}/bonding/slaves').split()
-            for member in self._members:
-                self.assertIn(member, slaves)
+            defined_mac = read_file(f'/sys/class/net/{interface}/bonding/ad_actor_system')
+            self.assertIn(defined_mac, system_mac)
 
     def test_bonding_evpn_multihoming(self):
         id = '5'