Skip to content

Commit

Permalink
mgr/dashboard: Support Description and AccountId in rgw roles
Browse files Browse the repository at this point in the history
Fixes https://tracker.ceph.com/issues/65506

Rgw roles list page is crashing due to these new fields coming from api but not supported in dashboard frontend.

This will also fix failing CI tests.

Signed-off-by: Afreen <afreen23.git@gmail.com>
  • Loading branch information
afreen23 committed Apr 16, 2024
1 parent 5f22a49 commit ab14968
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/pybind/mgr/dashboard/controllers/rgw.py
Original file line number Diff line number Diff line change
Expand Up @@ -858,9 +858,11 @@ def model(role_name: str):
"MaxSessionDuration": {'cellTemplate': 'duration'},
"RoleId": {'isHidden': True},
"AssumeRolePolicyDocument": {'isHidden': True},
"PermissionPolicies": {'isHidden': True}
"PermissionPolicies": {'isHidden': True},
"Description": {'isHidden': True},
"AccountId": {'isHidden': True}
},
detail_columns=['RoleId', 'AssumeRolePolicyDocument', 'PermissionPolicies'],
detail_columns=['RoleId', 'Description', 'AssumeRolePolicyDocument', 'PermissionPolicies', 'AccountId'],
meta=CRUDMeta()
)
class RgwUserRole(NamedTuple):
Expand All @@ -872,6 +874,8 @@ class RgwUserRole(NamedTuple):
MaxSessionDuration: int
AssumeRolePolicyDocument: str
PermissionPolicies: List
Description: str
AccountId: str


@APIRouter('/rgw/realm', Scope.RGW)
Expand Down

0 comments on commit ab14968

Please sign in to comment.