Petr Horáček has uploaded a new change for review.

Change subject: networkTest: UNKNOWN operstate
......................................................................

networkTest: UNKNOWN operstate

Sometimes kernel is not fast enough to write new device's
/sys/class/net/$devicename/operstate in time, in this case
originalState read by networkTests:nonChangingOperstate is
'UNKNOWN' and test which it uses fails.

Condition change: State change from 'UNKNOWN' to 'UP' is ok.

Change-Id: I6a5a059e717a331ae5e01cb64b7badc6757d4257
Signed-off-by: Petr Horáček <[email protected]>
---
M tests/functional/networkTests.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/87/30787/1

diff --git a/tests/functional/networkTests.py b/tests/functional/networkTests.py
index d509276..d286d5b 100644
--- a/tests/functional/networkTests.py
+++ b/tests/functional/networkTests.py
@@ -172,7 +172,8 @@
         changes = [(event.device, event.state) for event in monitor
                    if event.device == device]
         for _, state in changes:
-            if state != originalState:
+            if originalState != state and not (originalState == 'UNKNOWN' and
+                                               state == 'UP'):
                 raise OperStateChangedError('%s operstate changed: %s -> %r' %
                                             (device, originalState, changes))
 


-- 
To view, visit http://gerrit.ovirt.org/30787
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6a5a059e717a331ae5e01cb64b7badc6757d4257
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to