Hello Ohad Basan, Dan Kenigsberg,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/10314

to review the following change.

Change subject: restarting libvirtd didn't work over allinone setup
......................................................................

restarting libvirtd didn't work over allinone setup

Bug-Id: https://bugzilla.redhat.com/show_bug.cgi?id=888258
Change-Id: I1ab443fbef35f1ffaab59973f278eebe0143dd87
Signed-off-by: Yaniv Bronhaim <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/10237
Tested-by: Ohad Basan <[email protected]>
Reviewed-by: Ohad Basan <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M vdsm/vdsmd.init.in
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/14/10314/1

diff --git a/vdsm/vdsmd.init.in b/vdsm/vdsmd.init.in
index 7a7ae84..c8223d3 100755
--- a/vdsm/vdsmd.init.in
+++ b/vdsm/vdsmd.init.in
@@ -362,7 +362,11 @@
 
     ovirt_store_config "$lconf" "$qconf" "$ldconf" "$llogr"
 
-    /sbin/initctl restart libvirtd 2>/dev/null || :
+    if libvirt_should_use_upstart; then
+        /sbin/initctl restart libvirtd 2>/dev/null || :
+    else
+        /bin/systemctl restart libvirtd.service
+    fi
 
     #
     # finished reconfiguration, do not trigger


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1ab443fbef35f1ffaab59973f278eebe0143dd87
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.2
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Ohad Basan <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to