Allon Mureinik has uploaded a new change for review.

Change subject: core: Allow remove snapshot w/o disks (#825809)
......................................................................

core: Allow remove snapshot w/o disks (#825809)

https://bugzilla.redhat.com/825809

Some of the canDoAction validations in RemoveSnapshotCommand implicitly
assume that the snapshot contains disks.
This patch ignores these validations in case the snapshot does not
contain any disks, since they are not valid.

Change-Id: Ie65a93e75419b8b63e6c7e46fd9137f3db7684c4
Signed-off-by: Allon Mureinik <amure...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java
A 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RemoveSnapshotCommandTest.java
2 files changed, 109 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/19/6119/1
--
To view, visit http://gerrit.ovirt.org/6119
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie65a93e75419b8b63e6c7e46fd9137f3db7684c4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to