Arik Hadas has submitted this change and it was merged.

Change subject: core: extend DiskProfile.getAllForStorageDomain to a user query
......................................................................


core: extend DiskProfile.getAllForStorageDomain to a user query

getAllForStorageDomain is extended to a user query,
which will be consumed by the user portal as well.
In terms of MLA hierarchy, Disk profile is a under Storage Domain (SD),
so all the disk profiles attached to the SD will return if user has permission
to the SD, i.e. for we rely on SD permissions (with no change from current 
flow).

Change-Id: I3bb75728cd76bba7e39de2314275f5bc87942c6b
Signed-off-by: Gilad Chaplik <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/GetDiskProfilesByStorageDomainIdQuery.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/profiles/DiskProfileDao.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/profiles/DiskProfileDaoDbFacadeImpl.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/profiles/DiskProfileDaoTest.java
M packaging/dbscripts/disk_profiles_sp.sql
6 files changed, 43 insertions(+), 6 deletions(-)

Approvals:
  Gilad Chaplik: Verified
  Liron Aravot: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/29812
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3bb75728cd76bba7e39de2314275f5bc87942c6b
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to