Change in ovirt-engine[master]: core: Use container id for memory lock in import process.

mlipchuk at redhat.com mlipchuk at redhat.com
Thu Aug 22 15:28:50 UTC 2013


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Use container id for memory lock in import process.
......................................................................


core: Use container id for memory lock in import process.

When importing a VM we use memory lock on the VM in the export domain so
it will not get deleted in the process.

When the import process is being done with clone, a new vm id is generated
and the lock at the end command is being regenerated with the new
VM guid instead of the original one from the export domain.

The proposed fix is to use the container id from the parameters when
generating the shared lock.

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

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie849300bdc98020aeca90deeddafd12295e8211a
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas 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: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list