Skip to content

Commit

Permalink
dt/admin: Add node param to list_roles
Browse files Browse the repository at this point in the history
Useful for testing enterprise features, since a condition for
"RBAC enabled" is the presence of one or more non-default roles.

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
  • Loading branch information
oleiman committed Sep 17, 2024
1 parent d6e33cd commit 5fdc178
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tests/rptest/services/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -1165,15 +1165,16 @@ def delete_role(self, role: str, delete_acls: Optional[bool] = None):
def list_roles(self,
filter: Optional[str] = None,
principal: Optional[str] = None,
principal_type: Optional[str] = None):
principal_type: Optional[str] = None,
node=None):
params = {}
if filter is not None:
params['filter'] = filter
if principal is not None:
params['principal'] = principal
if principal_type is not None:
params['principal_type'] = principal_type
return self._request("get", "security/roles", params=params)
return self._request("get", "security/roles", params=params, node=node)

def update_role_members(self,
role: str,
Expand Down

0 comments on commit 5fdc178

Please sign in to comment.