Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-08 Thread Pavel Boldin 
Here you go: https://launchpad.net/~pboldin/+archive/ubuntu/libvirt-python

Use it, but please keep in mind that this is a draft reupload.

On Fri, Oct 2, 2015 at 11:38 PM, Mathieu Gagné  wrote:

> On 2015-10-02 4:18 PM, Pavel Boldin wrote:
> >
> > You have to pass device names from /dev/, e.g., if a VM has
> > ephemeral disk
> > attached at /dev/vdb you need to pass in 'vdb'. Format expected by
> > migrate_disks is ",...".
> >
> >
> > This is the format expected by the `virsh' utility and will not work in
> > Python.
> >
> > The libvirt-python has now support for passing lists to a parameter [1].
> >
> > [1]
> >
> http://libvirt.org/git/?p=libvirt-python.git;a=commit;h=9896626b8277e2ffba1523d2111c96b08fb799e8
> >
>
> Thanks for the info. I was banging my head against the wall, trying to
> understand why it didn't accept my list of strings.
>
> Now the next challenge is with Ubuntu packages, only python-libvirt
> 1.2.15 is available in Ubuntu Willy. :-/
>
> --
> Mathieu
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-08 Thread Mathieu Gagné 
Hi Pavel,

On 2015-10-08 9:39 AM, Pavel Boldin wrote:
> Here you go: https://launchpad.net/~pboldin/+archive/ubuntu/libvirt-python
> 
> Use it, but please keep in mind that this is a draft reupload.
> 

Thank you for your work. I'm sure a lot of people will benefit from it.
Live migration is a major keystone to our operation excellence and
anything that can improve it is welcomed.

I see the package is built against Wily. Will it work against Trusty? I
would say yes. It depends on libvirt0 (>= 1.2.16-2ubuntu11) which is
already available in the liberty repo of UCA.

Meanwhile, I managed to package 1.2.20 for test purposes for both Trusty
and Precise (not without raging =) as I needed to move on.

On a side note, I feel libvirt is such an important piece that it would
warrant its own official PPA. (both libvirt and libvirt-python)

I added comments to the review [1] because the change doesn't work in
its current state. I tested the patch against Kilo + Trusty + libvirt
1.2.20.

My main challenge at the moment is backporting the patch further to
Icehouse+Precise so I can move instances out of precise nodes and
upgrade them to trusty. Some far, and this is very early results, my
instances are stuck in "MIGRATING" state although they were completely
migrated to the destination. (domain is running on destination, removed
from source, ping is responding and I can SSH fine)

[1] https://review.openstack.org/#/c/227278/

-- 
Mathieu

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-05 Thread Daniel P. Berrange 
On Fri, Oct 02, 2015 at 04:38:30PM -0400, Mathieu Gagné wrote:
> On 2015-10-02 4:18 PM, Pavel Boldin wrote:
> > 
> > You have to pass device names from /dev/, e.g., if a VM has
> > ephemeral disk
> > attached at /dev/vdb you need to pass in 'vdb'. Format expected by
> > migrate_disks is ",...".
> > 
> > 
> > This is the format expected by the `virsh' utility and will not work in
> > Python.
> > 
> > The libvirt-python has now support for passing lists to a parameter [1].
> > 
> > [1]
> > http://libvirt.org/git/?p=libvirt-python.git;a=commit;h=9896626b8277e2ffba1523d2111c96b08fb799e8
> >  
> 
> Thanks for the info. I was banging my head against the wall, trying to
> understand why it didn't accept my list of strings.
> 
> Now the next challenge is with Ubuntu packages, only python-libvirt
> 1.2.15 is available in Ubuntu Willy. :-/

You already need a runtime version check for libvirt to see if the new
migration API is available. You just need to extend that to also check
the libvirt client version


Regards,
Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-05 Thread Pavel Boldin 
Daniel,

It is done already in the proposed patch.

But this one is about Wily having libvirt 1.2.16 and libvirt-python 1.2.15.

Pavel

On Mon, Oct 5, 2015 at 11:25 AM, Daniel P. Berrange 
wrote:

> On Fri, Oct 02, 2015 at 04:38:30PM -0400, Mathieu Gagné wrote:
> > On 2015-10-02 4:18 PM, Pavel Boldin wrote:
> > >
> > > You have to pass device names from /dev/, e.g., if a VM has
> > > ephemeral disk
> > > attached at /dev/vdb you need to pass in 'vdb'. Format expected by
> > > migrate_disks is ",...".
> > >
> > >
> > > This is the format expected by the `virsh' utility and will not work in
> > > Python.
> > >
> > > The libvirt-python has now support for passing lists to a parameter
> [1].
> > >
> > > [1]
> > >
> http://libvirt.org/git/?p=libvirt-python.git;a=commit;h=9896626b8277e2ffba1523d2111c96b08fb799e8
> > >
> >
> > Thanks for the info. I was banging my head against the wall, trying to
> > understand why it didn't accept my list of strings.
> >
> > Now the next challenge is with Ubuntu packages, only python-libvirt
> > 1.2.15 is available in Ubuntu Willy. :-/
>
> You already need a runtime version check for libvirt to see if the new
> migration API is available. You just need to extend that to also check
> the libvirt client version
>
>
> Regards,
> Daniel
> --
> |: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/
> :|
> |: http://libvirt.org  -o- http://virt-manager.org
> :|
> |: http://autobuild.org   -o- http://search.cpan.org/~danberr/
> :|
> |: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc
> :|
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-05 Thread Pavel Boldin 
OK, I propose to move further discussions of the topic to this particular
changeset review.

Pavel

On Mon, Oct 5, 2015 at 1:25 PM, Daniel P. Berrange 
wrote:

> On Mon, Oct 05, 2015 at 01:10:45PM +0300, Pavel Boldin wrote:
> > Daniel,
> >
> > It is done already in the proposed patch.
> >
> > But this one is about Wily having libvirt 1.2.16 and libvirt-python
> 1.2.15.
>
> Assuming you are refering to this patch:
>
> https://review.openstack.org/#/c/227278/3/nova/virt/libvirt/driver.py
>
> that code is only checking the libvirt daemon version, it is not checking
> the libvirt-python version.
>
>
> Regards,
> Daniel
> --
> |: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/
> :|
> |: http://libvirt.org  -o- http://virt-manager.org
> :|
> |: http://autobuild.org   -o- http://search.cpan.org/~danberr/
> :|
> |: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc
> :|
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-05 Thread Daniel P. Berrange 
On Mon, Oct 05, 2015 at 01:10:45PM +0300, Pavel Boldin wrote:
> Daniel,
> 
> It is done already in the proposed patch.
> 
> But this one is about Wily having libvirt 1.2.16 and libvirt-python 1.2.15.

Assuming you are refering to this patch:

https://review.openstack.org/#/c/227278/3/nova/virt/libvirt/driver.py

that code is only checking the libvirt daemon version, it is not checking
the libvirt-python version.


Regards,
Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-02 Thread Mathieu Gagné 
On 2015-10-02 4:18 PM, Pavel Boldin wrote:
> 
> You have to pass device names from /dev/, e.g., if a VM has
> ephemeral disk
> attached at /dev/vdb you need to pass in 'vdb'. Format expected by
> migrate_disks is ",...".
> 
> 
> This is the format expected by the `virsh' utility and will not work in
> Python.
> 
> The libvirt-python has now support for passing lists to a parameter [1].
> 
> [1]
> http://libvirt.org/git/?p=libvirt-python.git;a=commit;h=9896626b8277e2ffba1523d2111c96b08fb799e8
>  

Thanks for the info. I was banging my head against the wall, trying to
understand why it didn't accept my list of strings.

Now the next challenge is with Ubuntu packages, only python-libvirt
1.2.15 is available in Ubuntu Willy. :-/

-- 
Mathieu

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

[openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-02 Thread Pavel Boldin 
Resending, forgot to subscribe.



> You have to pass device names from /dev/, e.g., if a VM has ephemeral disk
> attached at /dev/vdb you need to pass in 'vdb'. Format expected by
> migrate_disks is ",...".
>

This is the format expected by the `virsh' utility and will not work in
Python.

The libvirt-python has now support for passing lists to a parameter [1].

[1]
http://libvirt.org/git/?p=libvirt-python.git;a=commit;h=9896626b8277e2ffba1523d2111c96b08fb799e8


Pavel
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Fwd: [nova] live migration in Mitaka

2015-10-02 Thread Pavel Boldin 
> Now the next challenge is with Ubuntu packages, only python-libvirt
> 1.2.15 is available in Ubuntu Willy. :-/
>
> Just create a PPA and build 1.2.17 from the git sources by copying the
`debian` directory from the package.

I can do it for you over the weekend if you want it.

Pavel
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev