Change in vdsm[master]: hack: disable NetworkManager on installation

danken at redhat.com danken at redhat.com
Tue Dec 18 09:49:03 UTC 2012


Dan Kenigsberg has uploaded a new change for review.

Change subject: hack: disable NetworkManager on installation
......................................................................

hack: disable NetworkManager on installation

When vdsm is told to own an interface (or bond) it takes it down, marks
it as NM_CONTROLLED=no, and takes it up again.

NetworkManager "unmanages" the device asynchronously, and then takes the
device down. Thus we may end up with a disconnected network
Until we find a smarter way to circuvent this unpredictable behavior
( https://bugzilla.redhat.com/879180 ), we have to turn off
NetworkManager when Vdsm is installed.

Change-Id: Ic248fb49711f991615abbb4debc0f1d91e47067b
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm.spec.in
M vdsm/vdsmd.service
2 files changed, 3 insertions(+), 1 deletion(-)


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

diff --git a/vdsm.spec.in b/vdsm.spec.in
index 68126fd..4826fcd 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -505,6 +505,7 @@
 %else
 if [ "$1" -eq 1 ] ; then
     /bin/systemctl enable vdsmd.service >/dev/null 2>&1 || :
+    /bin/systemctl --no-reload disable NetworkManager.service >/dev/null 2>&1 || :
     /bin/systemctl daemon-reload >/dev/null 2>&1 || :
 fi
 exit 0
@@ -542,6 +543,7 @@
 if [ "$1" -eq 0 ]; then
     /bin/systemctl --no-reload disable vdsmd.service > /dev/null 2>&1 || :
     /bin/systemctl stop vdsmd.service > /dev/null 2>&1 || :
+    /bin/systemctl enable NetworkManager.service > /dev/null 2>&1 || :
 fi
 exit 0
 %endif
diff --git a/vdsm/vdsmd.service b/vdsm/vdsmd.service
index 6a650f4..01224dc 100644
--- a/vdsm/vdsmd.service
+++ b/vdsm/vdsmd.service
@@ -1,7 +1,7 @@
 [Unit]
 Description=Virtual Desktop Server Manager
 Requires=multipathd.service libvirtd.service ntpd.service
-Conflicts=libvirt-guests.service
+Conflicts=libvirt-guests.service NetworkManager.service
 
 [Service]
 Type=forking


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic248fb49711f991615abbb4debc0f1d91e47067b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list