Omer Frenkel has submitted this change and it was merged. Change subject: core: change RemoveVmFromImportExportParamenters to get VM ID ......................................................................
core: change RemoveVmFromImportExportParamenters to get VM ID RemoveVmFromImportExportCommand only needs the ID of the VM to work with, yet RemoveVmFromImportExportParamenters received VM object. at the REST API and at the UI we were creating VM object just as a wrapper of the VM ID to pass to the parameters class. This patch changes the parameter class to receive ID of VM instead of VM object. This patch also contains minor refactoring in RemoveVmFromImportExportParamenters class. Change-Id: I0fff20548248d3c47087d2d004d285c429a1c081 Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RemoveVmFromImportExportParamenters.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainVmsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainVmsResourceTest.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/VmBackupModel.java 4 files changed, 17 insertions(+), 28 deletions(-) Approvals: Michael Pasternak: Looks good to me, but someone else must approve Omer Frenkel: Verified; Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/16707 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0fff20548248d3c47087d2d004d285c429a1c081 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Michael Pasternak <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
