Change in vdsm[master]: network: tests do not check OVS devices operstate

phoracek at redhat.com phoracek at redhat.com
Tue Feb 9 13:30:52 UTC 2016


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

Change subject: network: tests do not check OVS devices operstate
......................................................................

network: tests do not check OVS devices operstate

We have to fake not only waitForKnownState, but also waitForState,
because of tests expects bond to turn UP, but that never happens.

Change-Id: I283ec884e9b9844e1d3ce59b5809d08f9c2f01fc
Signed-off-by: Petr Horáček <phoracek at redhat.com>
---
M tests/functional/networkTestsOVS.py
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/30/53130/1

diff --git a/tests/functional/networkTestsOVS.py b/tests/functional/networkTestsOVS.py
index db7aaa8..868fdf5 100644
--- a/tests/functional/networkTestsOVS.py
+++ b/tests/functional/networkTestsOVS.py
@@ -143,9 +143,10 @@
 
 
 # When we set OVS bond device up, it does not turn UP, but only UNKNOWN
-def _fakeWaitForKnownOperstate(*args, **kwargs):
+def _fakeWaitForOperstate(*args, **kwargs):
     pass
-networkTests._waitForKnownOperstate = _fakeWaitForKnownOperstate
+networkTests._waitForOperstate = _fakeWaitForOperstate
+networkTests._waitForKnownOperstate = _fakeWaitForOperstate
 
 
 class OVSVdsProxy(_VdsProxy):


-- 
To view, visit https://gerrit.ovirt.org/53130
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I283ec884e9b9844e1d3ce59b5809d08f9c2f01fc
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list