Moti Asayag has submitted this change and it was merged.

Change subject: core: Add queries for fetching Permitted Storage Domains
......................................................................


core: Add queries for fetching Permitted Storage Domains

The patch adds two queries for fetching Storage Domains:

GetPermittedStorageDomainsByStoragePoolId(StoragePoolID, ActionGroup) -
returns all storage domains within the specified storage-pool-id that
the current user has permissions on with a role that contains the
specified action-group.

GetPermittedStorageDomainsByTemplateId(TemplateID, ActionGroup) -
returns all storage domains on which the specified template resides that
the current user has permissions on with a role that contains the
specified action-group.

The patch refactors GetPermittedStorageDomainsByTemplateIdQuery so it
could be easily inherited by the new query.

Change-Id: I155cb9f187b951b399e99b54007cf24e3163251a
---
M backend/manager/dbscripts/storages_sp.sql
A 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetPermittedStorageDomainsByStoragePoolIdQuery.java
A 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetPermittedStorageDomainsByTemplateIdQuery.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetStorageDomainsByVmTemplateIdQuery.java
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetEntitiesWithPermittedActionParameters.java
A 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetPermittedStorageDomainsByStoragePoolIdParameters.java
A 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetPermittedStorageDomainsByTemplateIdParameters.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/StorageDomainDAO.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainDAODbFacadeImpl.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainDAOWrapperImpl.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/PermissionDAOTest.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDAOTest.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
14 files changed, 228 insertions(+), 14 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I155cb9f187b951b399e99b54007cf24e3163251a
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to