From Dan Kenigsberg <danken(a)redhat.com>:
Hello Leon Goldberg,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/70180
to review the following change.
Change subject: check-merged: experimenting functional tests without NM
......................................................................
check-merged: experimenting functional tests without NM
We now know that NM causes functional tests to be slow,
and for several of them - to fail.
this patch disabled NM in order to make check-merged job
pass on 4.1 branch as it does on master.
Change-Id: I42608420b42f389fdf3981a22ec4d2077904474d
Signed-off-by: Leon Goldberg <lgoldber(a)redhat.com>
---
M automation/check-merged.sh
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/80/70180/1
diff --git a/automation/check-merged.sh b/automation/check-merged.sh
index 3ec470d..a52a209 100755
--- a/automation/check-merged.sh
+++ b/automation/check-merged.sh
@@ -72,6 +72,8 @@
local res=0
lago shell "$vm_name" -c \
" \
+ systemctl stop NetworkManager
+ systemctl mask NetworkManager
cd /usr/share/vdsm/tests
./run_tests.sh \
-a type=functional,switch=legacy \
--
To view, visit
https://gerrit.ovirt.org/70180
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I42608420b42f389fdf3981a22ec4d2077904474d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>