Change in ovirt-engine[master]: core: Permission based icon filtering: backend part

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Sep 21 14:58:12 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: core: Permission based icon filtering: backend part
......................................................................


core: Permission based icon filtering: backend part

I allows to non-admin user to list icons using REST. Returned icons are
union of predefined icons and icons that are associated with some
entity the user has permission to (vm or tempate).

Change-Id: I09aecac8c8265cf687ecf4edc2305421a2feb61b
Bug-Url: https://bugzilla.redhat.com/1263758
Signed-off-by: Jakub Niedermertl <jniederm at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmIconsQuery.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmIconDao.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmIconDaoImpl.java
4 files changed, 29 insertions(+), 7 deletions(-)

Approvals:
  Jakub Niedermertl: Verified
  Jenkins CI: Passed CI tests
  Omer Frenkel: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/46246
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I09aecac8c8265cf687ecf4edc2305421a2feb61b
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list