Moti Asayag has posted comments on this change. Change subject: BZ#811807 Define network filter on libvirt ......................................................................
Patch Set 1: (1 inline comment) Tools isn't good enough since it fails to establish the libvirt connection. In order to achieve it some additional work should be made. The error is: File "/usr/lib64/python2.6/site-packages/vdsm/libvirtconnection.py", line 25, in <module> import libvirtev ImportError: No module named libvirtev .................................................... File vdsm/vdsmd.init.in Line 452: fi Line 453: sleep 1 Line 454: done Line 455: # End workaround Line 456: This is required specifically now since we attempt to connect to libvirt immediately after the libvirt service is started. It takes 2-3 seconds for the service to become responsive. Without this workaround, first attempt to configure nwfilter will fail. I'll add this part as a separate patch, at the head of this patch series. Line 457: return 0 Line 458: else Line 459: echo "$startout" >&2 Line 460: return 1 -- To view, visit http://gerrit.ovirt.org/7354 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9f1708385dec6a87bc404e4ab25c4da8ab8a8acc Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masa...@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