Allon Mureinik has posted comments on this change. Change subject: database: Define user roles to import/export domains ......................................................................
Patch Set 1: Code-Review+1 (2 comments) Looks about right to me. Eli? https://gerrit.ovirt.org/#/c/41055/1//COMMIT_MSG Commit Message: Line 10: assigned ActionGroup.IMPORT_EXPORT_VM to these roles. This change is required Line 11: in order to enable users, using users portal, to import/export VMs or Line 12: Templates. Line 13: In addition, assigned ActionGroup.IMPORT_EXPORT_VM to UserVmManager, Line 14: PowerUserRole and TemplateOwner roles. Worth noting that users who are granted this role could ALREADY export/import via REST API, regardless of following patches. Line 15: Line 16: Change-Id: I61837fdc2fe2eae50b486dad5fbfeeac42c472f0 Line 11: in order to enable users, using users portal, to import/export VMs or Line 12: Templates. Line 13: In addition, assigned ActionGroup.IMPORT_EXPORT_VM to UserVmManager, Line 14: PowerUserRole and TemplateOwner roles. Line 15: Missing "Bug-Url:" Line 16: Change-Id: I61837fdc2fe2eae50b486dad5fbfeeac42c472f0 -- To view, visit https://gerrit.ovirt.org/41055 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I61837fdc2fe2eae50b486dad5fbfeeac42c472f0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
