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

Reply via email to