Libor Spevak has posted comments on this change.

Change subject: engine: Do not add disk in snapshot preview
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachDiskToVmCommand.java
Line 99:         final boolean isNotDuringSnapshot =
Line 100:                 
validate(getSnapshotsValidator().vmNotDuringSnapshot(getVmId())) &&
Line 101:                 
validate(getSnapshotsValidator().vmNotInPreview(getVmId()));
Line 102: 
Line 103:         if (isImageDisk && !isNotDuringSnapshot) {
Ok, I remove isImageDisk condition. So attaching any kind of disk will not be 
possible.
Line 104:             return false;
Line 105:         }
Line 106: 
Line 107:         if (getParameters().isPlugUnPlug()


--
To view, visit http://gerrit.ovirt.org/13327
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibfde0540f1d77456ed1412b015a0963f4682039f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Libor Spevak <lspe...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Libor Spevak <lspe...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to