----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19730/#review38833 -----------------------------------------------------------
will commit this. It works in our environment because of our /etc/libvirt/qemu.conf, I suppose. - Marcus Sorensen On March 27, 2014, 5:53 p.m., Yoshikazu Nojima wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19730/ > ----------------------------------------------------------- > > (Updated March 27, 2014, 5:53 p.m.) > > > Review request for cloudstack and Marcus Sorensen. > > > Bugs: CLOUDSTACK-6292 > https://issues.apache.org/jira/browse/CLOUDSTACK-6292 > > > Repository: cloudstack-git > > > Description > ------- > > When migrating VM, VNC server IP address in domain xml need to be updated to > destination KVM host IP address, but updated domain XML is left unused. > This patch changes MigrateKVMAsync class to accept domXml. > > > Diffs > ----- > > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java > 1d7d1b2 > > Diff: https://reviews.apache.org/r/19730/diff/ > > > Testing > ------- > > Live migration between two KVM hosts. > > > Thanks, > > Yoshikazu Nojima > >
