Michael Kublin has submitted this change and it was merged. Change subject: core : Clean up for MoveVm because of introducing multiple storage domains ......................................................................
core : Clean up for MoveVm because of introducing multiple storage domains The following patch should clean MoveVmCommand 1. Remove wrong code 2. Change an API for moving disks 3. Remove unused code 4. Command was marked as deprecated, because it is still used at rest Change-Id: Id6714e2d09c703de7e3ed4b9c697468703f2e5c2 --- D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveMultipleImageGroupsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveVmCommand.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/MoveMultipleImageGroupsParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/MoveOrCopyParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/asynctasks/AsyncTaskType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/job/StepEnum.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/MoveMultipleImageGroupsVDSCommandParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonMoveOrCopyParametersMixIn.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonObjectDeserializer.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonObjectSerializer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IIrsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerWrapper.java D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/MoveMultipleImageGroupsVDSCommand.java 17 files changed, 38 insertions(+), 354 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/3617 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6714e2d09c703de7e3ed4b9c697468703f2e5c2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches