You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by sw...@apache.org on 2016/04/28 21:55:56 UTC

[2/2] git commit: updated refs/heads/4.7 to c03e359

Merge pull request #1348 from DaanHoogland/CLOUDSTACK-9142

CLOUDSTACK-9142 Migrate VM changes xmlDesc in a safe wayThe problem arises when the origin hypervisor has an ip addres that ends with 1, like '10.10.10.1' and the qemu VM description is containing an address that has that as part of its address, '10.10.10.100' for instance.
now migrating to '10.10.10.10' will change both addresses in the xml description file for qemu. It is fixed and unit tests are added. I am not sure yet how to integration test this. Regression will probably work so creating a PR now.

* pr/1348:
  CLOUDSTACK-9142 Migrate VM changes xmlDesc in a safe way

Signed-off-by: Will Stevens <wi...@gmail.com>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/c03e3598
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/c03e3598
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/c03e3598

Branch: refs/heads/4.7
Commit: c03e359824327b56af6b1921155d8796d2958f9c
Parents: d13ff88 8a6afa6
Author: Will Stevens <wi...@gmail.com>
Authored: Thu Apr 28 15:55:09 2016 -0400
Committer: Will Stevens <wi...@gmail.com>
Committed: Thu Apr 28 15:55:09 2016 -0400

----------------------------------------------------------------------
 .../wrapper/LibvirtMigrateCommandWrapper.java   |  34 ++-
 .../resource/LibvirtComputingResourceTest.java  |  22 +-
 .../LibvirtMigrateCommandWrapperTest.java       | 306 +++++++++++++++++++
 3 files changed, 349 insertions(+), 13 deletions(-)
----------------------------------------------------------------------