Tal Nisan has uploaded a new change for review. Change subject: core: Check attached VMs snapshot preview only status when moving a disk ......................................................................
core: Check attached VMs snapshot preview only status when moving a disk Change-Id: Ibe4de57b5e0dabcb28320a305dd32bbfb705a623 Signed-off-by: Tal Nisan <tni...@redhat.com> Bug-Url: https://bugzilla.redhat.com/1211330 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java 1 file changed, 1 insertion(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/93/40593/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java index 007e690..96f7bb8 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java @@ -188,8 +188,7 @@ VmDevice vmDevice = pair.getSecond(); if (vmDevice.getSnapshotId() == null) { // Skip check for VMs with connected snapshot VM vm = pair.getFirst(); - if (!validate(snapshotsValidator.vmNotInPreview(vm.getId())) - || !validate(snapshotsValidator.vmNotDuringSnapshot(vm.getId()))) { + if (!validate(snapshotsValidator.vmNotInPreview(vm.getId()))) { return false; } } -- To view, visit https://gerrit.ovirt.org/40593 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ibe4de57b5e0dabcb28320a305dd32bbfb705a623 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tni...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches