Roy Golan has submitted this change and it was merged.
Change subject: core: Added cpu profiles permissions propagation
......................................................................
core: Added cpu profiles permissions propagation
Cpu Profile permissions will now propagate from parent objects of Cpu
Profile.
For example, If you have CpuProfileOperator permissions on a cluster
that has Cpu Profile 'A' then you will automatically have
CpuProfileOperator on 'A' and all the other Cpu Profiles on that cluster.
The list of objects that Cpu Profile permissions progpagates from is:
System -> Storage Domain -> Cluster -> Cpu Profile.
This also solves the problem that admin user (That has SuperUser
permissions on System) couldn't run any virtual machines because it
needed to have specific permissions for a CpuProfile on it's cluster.
Change-Id: Iaf5c142f8e563377cc262bf36ea5a44060b90d81
Bug-Url:
https://bugzilla.redhat.com/1143869
Signed-off-by: Tomer Saban <tsaban(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/CpuProfileHelper.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/GetCpuProfilesByClusterIdQuery.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/profiles/CpuProfileDao.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/profiles/CpuProfileDaoImpl.java
M
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/profiles/CpuProfileDaoTest.java
M packaging/dbscripts/cpu_profiles_sp.sql
M packaging/dbscripts/create_views.sql
7 files changed, 66 insertions(+), 32 deletions(-)
Approvals:
Eli Mesika: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Tomer Saban: Verified
Allon Mureinik: Looks good to me, but someone else must approve
Roy Golan: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/52727
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf5c142f8e563377cc262bf36ea5a44060b90d81
Gerrit-PatchSet: 14
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>