
On Tue, Sep 25, 2012 at 03:45:24PM +0800, liguang wrote:
original migration did not aware of offline case, so, try to support offline migration quietly (did not disturb original migration) by pass VIR_MIGRATE_OFFLINE flag to migration APIs if only the domain is really inactive, and migration process will not puzzled by domain offline and exit unexpectedly. these changes did not take care of disk images the domain required, for them could be transferred by other APIs as suggested, then VIR_MIGRATE_OFFLINE should not combined with VIR_MIGRATE_NON_SHARED_*. so, this migration result is just make domain definition alive at target side.
Signed-off-by: liguang <lig.fnst@cn.fujitsu.com> --- include/libvirt/libvirt.h.in | 1 + src/qemu/qemu_driver.c | 15 ++++++++++++ src/qemu/qemu_migration.c | 53 ++++++++++++++++++++++++++++++++++++----- src/qemu/qemu_migration.h | 3 +- tools/virsh-domain.c | 6 ++++ 5 files changed, 70 insertions(+), 8 deletions(-)
In my testing this patch is still badly broken, apparently no different than the previous version I tested I have 2 hosts, avocado & mustard and stopped guest vm1. I try to migrate the VM from avocado to mustard thus: # ./tools/virsh -c qemu+ssh://root@avocado/system migrate --undefinesource --offline vm1 qemu+ssh://root@mustard/system This should *not* even run because I did not pass the --persistent flag, but it does in fact complete. I creates a guest 'vm1' on the target host, but does not write out any config. This is a completely bogus scenario. If a guest has no config, it is transient and thus must not exist if it is shutoff, but your patch is resulting in a transient, shutoff guest. If I set the '--persistent' flag, there is no diffrence in behaviour. We still get a transient shutoff guest on the target. Also the '--undefinesource' flag appeared to have no effect at all. The guest still exists on the source, both in libvirtd & on disk. 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 :|