Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: vdsmd.init: verify if libvirt socket file exists
......................................................................

vdsmd.init: verify if libvirt socket file exists

Currently, VDSM init script checks if libvirt daemon is up with pgrep command.
However, libvirt daemon shows up before the socket file be ready.

This patch will replace the validation from pgrep to a validation which checks 
if
the socket file is really exists.

Change-Id: I0ada84192a639c7780c40d7df62e22e3c9fa9e16
Bug-Url: https://bugzilla.redhat.com/859483
Signed-off-by: Douglas Schilling Landgraf <[email protected]>
---
M vdsm/vdsmd.init.in
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/75/8175/1

diff --git a/vdsm/vdsmd.init.in b/vdsm/vdsmd.init.in
index e32c3e5..23ee7d3 100755
--- a/vdsm/vdsmd.init.in
+++ b/vdsm/vdsmd.init.in
@@ -402,8 +402,7 @@
 
     for i in {1..50}
     do
-        if pgrep libvirtd > /dev/null 2>&1;
-        then
+        if [ -S /var/run/libvirt/libvirt-sock ]; then
             return 0
         fi
         sleep 0.2


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0ada84192a639c7780c40d7df62e22e3c9fa9e16
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to