Change in ovirt-engine[master]: core: Sync VM disks with new guid when importing a VM.

mlipchuk at redhat.com mlipchuk at redhat.com
Tue Jul 23 12:34:57 UTC 2013


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Sync VM disks with new guid when importing a VM.
......................................................................


core: Sync VM disks with new guid when importing a VM.

Currently ImportVmCommand use three different kinds of containers when
importing a VM: image id, disk id, and the new disk which is about to be
added to the DB.

The use of those containers has dependency with each other,
and should be reflected in only one container so it will remain in sync.

the following fix, put those parameters into one map, and use this map
when copying a new disk into the Data Center.

Change-Id: I5e25c75b95e08bbb5955b4d96a17baff59336e68
Bug-Url: https://bugzilla.redhat.com/970485
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
1 file changed, 40 insertions(+), 19 deletions(-)

Approvals:
  Maor Lipchuk: Verified; Looks good to me, approved


-- 
To view, visit http://gerrit.ovirt.org/17169
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5e25c75b95e08bbb5955b4d96a17baff59336e68
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list