On Mon, May 03, 2010 at 09:52:10AM -0400, Cole Robinson wrote:
On 05/03/2010 04:13 AM, Kenneth Nagin wrote:
>
...
> This the patch file:
> (See attached file: libvirt_migration_ns_100503.patch)
>
ACK, patch looks fine now.
No objections to the patch that was committed, but FYI there is an
improvement that can be done to it.
If you run 'virsh domjobinfo $GUEST' while migration is running it
tells you the progress of migration. It gives overall progress, and
memory progress. The APi was designed to allow us to report storage
progress too. To wire this up look at the method
qemuDomainWaitForMigrationComplete
where it updates the priv->jobInfo variables. You'll also need to add
extra params to the qemuMonitorGetMigrationStatus() method to return
the disk processed/remaining/total stats
virsh should already be able to report this data once the QEMU driver
is wired up
Regards,
Daniel
--
|: Red Hat, Engineering, London -o-
http://people.redhat.com/berrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org -o-
http://deltacloud.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|