Change in ovirt-engine[ovirt-engine-3.5]: core: extend DiskProfile.getAllForStorageDomain to a user query

tnisan at redhat.com tnisan at redhat.com
Mon Aug 25 08:14:28 UTC 2014


Tal Nisan 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
Bug-Url: https://bugzilla.redhat.com/1085049
Signed-off-by: Gilad Chaplik <gchaplik at redhat.com>
---
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; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3bb75728cd76bba7e39de2314275f5bc87942c6b
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list