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 <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: Xavi Francisco <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
