]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test-netowrk: in test_bridge_configure_without_carrier, avoid strange test failures 16411/head
authorDan Streetman <ddstreet@canonical.com>
Wed, 8 Jul 2020 21:05:03 +0000 (17:05 -0400)
committerDan Streetman <ddstreet@canonical.com>
Wed, 8 Jul 2020 21:05:03 +0000 (17:05 -0400)
For some reason the wait-online is failing intermittently; it's unclear
exactly why, but this hopefully avoids the failure for unrelated PR.

This is a workaround (not fix) for #16105

test/test-network/systemd-networkd-tests.py

index bb436e8bce5800da6d6b6f5dbe0ca6d6966929a4..4dca56f7dcb86ae441a10b16982dbffe9ef536ae 100755 (executable)
@@ -2879,8 +2879,7 @@ class NetworkdBridgeTests(unittest.TestCase, Utilities):
             with self.subTest(test=test):
                 if test == 'no-slave':
                     # bridge has no slaves; it's up but *might* not have carrier
-                    # It may take very long time that the interface become configured state.
-                    self.wait_online(['bridge99:no-carrier'], timeout='2m', setup_state=None)
+                    self.wait_operstate('bridge99', operstate=r'(no-carrier|routable)', setup_state=None, setup_timeout=30)
                     # due to a bug in the kernel, newly-created bridges are brought up
                     # *with* carrier, unless they have had any setting changed; e.g.
                     # their mac set, priority set, etc.  Then, they will lose carrier
@@ -2891,7 +2890,7 @@ class NetworkdBridgeTests(unittest.TestCase, Utilities):
                     # add slave to bridge, but leave it down; bridge is definitely no-carrier
                     self.check_link_attr('test1', 'operstate', 'down')
                     check_output('ip link set dev test1 master bridge99')
-                    self.wait_online(['bridge99:no-carrier:no-carrier'], setup_state=None)
+                    self.wait_operstate('bridge99', operstate='no-carrier', setup_state=None)
                     self.check_link_attr('bridge99', 'carrier', '0')
                 elif test == 'slave-up':
                     # bring up slave, which will have carrier; bridge gains carrier