Change in ovirt-engine[ovirt-engine-3.6]: core: fix import from configuration as clone

tnisan at redhat.com tnisan at redhat.com
Mon Apr 11 10:59:27 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: fix import from configuration as clone
......................................................................


core: fix import from configuration as clone

Apparently, when importing a VM as a new entity (clone) the REST-API
client does not necessarily set the ID of the VM in the parameters
(not as part of the VM object) and recent changes done in the import
command cause the command to ignore the request for cloning the VM
in this case.

Therefore, the decision whether to import a VM as clone is changed
to include the case that the import-as-new-entity flag is set and
the ID of the VM is not set in the parameters as well.

Change-Id: Ia750fda289f2a8da3deddce273cc15f51a062a44
Bug-Url: https://bugzilla.redhat.com/1322602
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/55933
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia750fda289f2a8da3deddce273cc15f51a062a44
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list