Liron Ar has posted comments on this change. Change subject: core: ISO Storage Domain can be detached for suspended VM. ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/26309/1//COMMIT_MSG Commit Message: Line 6: Line 7: core: ISO Storage Domain can be detached for suspended VM. Line 8: Line 9: Since the VM process is not running when the VM is suspended, we should Line 10: allow the user to detach the ISO Storage Domain, even if the VM is > 1. done 2. how it's like vm down? on vm down we should check when starting that the iso domain is active, in this the vm already uses it. 3. same 5. it's not checked on detach. Line 11: suspended. Line 12: Line 13: Change-Id: I3d6c75fccada58d9a537c94040b0eb86b3332e3f -- To view, visit http://gerrit.ovirt.org/26309 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3d6c75fccada58d9a537c94040b0eb86b3332e3f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Liron Ar <lara...@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com> Gerrit-Reviewer: Xavi Francisco <xfran...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches