Change in vdsm[master]: vdsm: Use virsh to check if libvirt is up

masayag at redhat.com masayag at redhat.com
Thu Sep 27 13:15:35 UTC 2012


Moti Asayag has uploaded a new change for review.

Change subject: vdsm: Use virsh to check if libvirt is up
......................................................................

vdsm: Use virsh to check if libvirt is up

The patch replaces libvirt socket file existence check with a check
using virsh, so there is no need to depend on internal libvirt
implementation details.

Bug-Url: https://bugzilla.redhat.com/859483

Change-Id: Ic2844070ca0816525851b3b495b3dfe3096058a9
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M vdsm/vdsmd.init.in
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/45/8245/1

diff --git a/vdsm/vdsmd.init.in b/vdsm/vdsmd.init.in
index f1249b6..1503da1 100755
--- a/vdsm/vdsmd.init.in
+++ b/vdsm/vdsmd.init.in
@@ -402,7 +402,7 @@
 
     for i in {1..50}
     do
-        if [ -S /var/run/libvirt/libvirt-sock ] &&
+        if virsh -r version > /dev/null 2>&1 &&
                 pgrep libvirtd > /dev/null 2>&1; then
             return 0
         fi


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic2844070ca0816525851b3b495b3dfe3096058a9
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>


More information about the vdsm-patches mailing list