Daniel Erez has posted comments on this change. Change subject: core: AddVmTemplate - check perms on target SDs ......................................................................
Patch Set 1: Omer - according to the patch, creating a template without disks still requires permissions on the datacenter (as today). But actually, shouldn't the container be cluster instead of datacenter? Otherwise, if we need permission on the datacenter regardless, then the current situation could be good enough and no fix is needed.. -- To view, visit http://gerrit.ovirt.org/32199 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib6b7913a6a456e8df1b8962a96757c4f77e0939f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
