Change in vdsm[master]: BZ#811807 Libvirt service engagement workaround

masayag at redhat.com masayag at redhat.com
Tue Aug 21 12:38:33 UTC 2012


Moti Asayag has posted comments on this change.

Change subject: BZ#811807 Libvirt service engagement workaround
......................................................................


Patch Set 1: (4 inline comments)

....................................................
File vdsm/vdsmd.init.in
Line 442:     then
Line 443: 
Line 444:         # Begin workaround
Line 445:         # Libvirt reports its service status as responsive despite its socket
Line 446:         # is not yet ready. Once issue fixed on libvirt, this WA shouls be removed.
Done
Line 447:         for i in {1..20}
Line 448:         do
Line 449:             if pgrep libvirtd > /dev/null 2>&1;
Line 450:             then


Line 442:     then
Line 443: 
Line 444:         # Begin workaround
Line 445:         # Libvirt reports its service status as responsive despite its socket
Line 446:         # is not yet ready. Once issue fixed on libvirt, this WA shouls be removed.
Done
Line 447:         for i in {1..20}
Line 448:         do
Line 449:             if pgrep libvirtd > /dev/null 2>&1;
Line 450:             then


Line 449:             if pgrep libvirtd > /dev/null 2>&1;
Line 450:             then
Line 451:                 return 0
Line 452:             fi
Line 453:             sleep 1
After several tests it takes a second or less for pgrep to report libvirt process.
will modify it to 0.2, and will shorten the range for 10 seconds.
Line 454:         done
Line 455:         # End workaround
Line 456: 
Line 457:         return 0


Line 450:             then
Line 451:                 return 0
Line 452:             fi
Line 453:             sleep 1
Line 454:         done
will issue a message to log if our 10 seconds timeout expired.
Line 455:         # End workaround
Line 456: 
Line 457:         return 0
Line 458:     else


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7f6fb1ef12caeca3a47adec206b653932026634f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list