Dan Kenigsberg has posted comments on this change. Change subject: BZ#811807 Libvirt service engagement workaround ......................................................................
Patch Set 2: Looks good to me, but someone else must approve (1 inline comment) .................................................... File vdsm/vdsmd.init.in Line 459: startout=`/sbin/initctl start libvirtd 2>&1` Line 460: if [[ "$?" -eq 0 || "$startout" =~ .*already\ running.* ]]; Line 461: then Line 462: Line 463: # Begin workaround frankly, I believe that a good name for the function would have made these comments needless. Line 464: await_libvirt_start Line 465: # End workaround Line 466: Line 467: return 0 -- 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: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Igor Lvovsky <ilvov...@redhat.com> Gerrit-Reviewer: Livnat Peer <lp...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches