<div dir="ltr"><div>OK, I propose to move further discussions of the topic to this particular changeset review.<br><br></div>Pavel<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 5, 2015 at 1:25 PM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Oct 05, 2015 at 01:10:45PM +0300, Pavel Boldin wrote:<br>
> Daniel,<br>
><br>
> It is done already in the proposed patch.<br>
><br>
> But this one is about Wily having libvirt 1.2.16 and libvirt-python 1.2.15.<br>
<br>
</span>Assuming you are refering to this patch:<br>
<br>
<a href="https://review.openstack.org/#/c/227278/3/nova/virt/libvirt/driver.py" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/227278/3/nova/virt/libvirt/driver.py</a><br>
<br>
that code is only checking the libvirt daemon version, it is not checking<br>
the libvirt-python version.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Regards,<br>
Daniel<br>
--<br>
|: <a href="http://berrange.com" rel="noreferrer" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" rel="noreferrer" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" rel="noreferrer" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" rel="noreferrer" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" rel="noreferrer" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/~danberr/" rel="noreferrer" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" rel="noreferrer" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" rel="noreferrer" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</div></div></blockquote></div><br></div>