Remove unused hints from assignment APIs
The controller is responsible for listing user projects based on role assignments and would build a hints objects and pass it to the assignment manager. This is a common pattern used across keystone's APIs. But, the assignment API never actually passed the hints objects to the backend implementation. This commit removes the hints from being passed to the manager for list_projects_for_user and list_domains_for_user because those APIs never use the hints object. This should allow us to implement caching to speed up those calls later. Change-Id: I9b1c8c30ca6a78dd6e78add7de278e467ceea046 Related-Bug: 1700852
This commit is contained in:
parent
86b4b4a747
commit
9fccd38d1b
@ -266,8 +266,7 @@ class ProjectAssignmentV3(controller.V3Controller):
|
||||
@controller.filterprotected('domain_id', 'enabled', 'name')
|
||||
def list_user_projects(self, request, filters, user_id):
|
||||
hints = ProjectAssignmentV3.build_driver_hints(request, filters)
|
||||
refs = self.assignment_api.list_projects_for_user(user_id,
|
||||
hints=hints)
|
||||
refs = self.assignment_api.list_projects_for_user(user_id)
|
||||
return ProjectAssignmentV3.wrap_collection(request.context_dict,
|
||||
refs,
|
||||
hints=hints)
|
||||
|
@ -223,7 +223,7 @@ class Manager(manager.Manager):
|
||||
|
||||
# TODO(henry-nash): We might want to consider list limiting this at some
|
||||
# point in the future.
|
||||
def list_projects_for_user(self, user_id, hints=None):
|
||||
def list_projects_for_user(self, user_id):
|
||||
assignment_list = self.list_role_assignments(
|
||||
user_id=user_id, effective=True)
|
||||
# Use set() to process the list to remove any duplicates
|
||||
@ -233,7 +233,7 @@ class Manager(manager.Manager):
|
||||
|
||||
# TODO(henry-nash): We might want to consider list limiting this at some
|
||||
# point in the future.
|
||||
def list_domains_for_user(self, user_id, hints=None):
|
||||
def list_domains_for_user(self, user_id):
|
||||
assignment_list = self.list_role_assignments(
|
||||
user_id=user_id, effective=True)
|
||||
# Use set() to process the list to remove any duplicates
|
||||
|
Loading…
Reference in New Issue
Block a user