From Dan Kenigsberg <dan...@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 <lgold...@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 <dan...@redhat.com>
Gerrit-Reviewer: Leon Goldberg <lgold...@redhat.com>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to