Skip to content

[CLOUDGA-33268] Use UserDataWithRoleInfo and role_info for user listing - #327

Merged
bansal01yash merged 1 commit into
mainfrom
CLOUDGA-33268-cli-role-info
Apr 20, 2026
Merged

[CLOUDGA-33268] Use UserDataWithRoleInfo and role_info for user listing#327
bansal01yash merged 1 commit into
mainfrom
CLOUDGA-33268-cli-role-info

Conversation

@bansal01yash

@bansal01yash bansal01yash commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update Go client dependency to consume the new listAccountUsers response type (UserListWithRoleResponse / UserDataWithRoleInfo) from #24491
  • Read role display names from the new lightweight role_info field instead of the deeply nested role_list chain (Info.GetRoleList()[0].GetRoles()[0].Info.GetDisplayName())
  • Update GetUserByEmail return type to UserDataWithRoleInfo

Test plan

  • Run ybm user list and verify it still works

Note

Go client points to branch CLOUDGA-33268-role-info-client on yugabytedb-managed-go-client-internal. A follow-up PR will update to the stable master

@bansal01yash
bansal01yash requested a review from a team as a code owner April 20, 2026 10:23
Comment thread internal/formatter/users.go Outdated
The listAccountUsers API now returns UserListWithRoleResponse with
UserDataWithRoleInfo, which includes a lightweight role_info field.
Update the CLI to consume the new response type and read role display
names from role_info instead of the nested role_list chain.
@bansal01yash
bansal01yash force-pushed the CLOUDGA-33268-cli-role-info branch from 52f81c3 to c35d2d7 Compare April 20, 2026 12:14
@bansal01yash
bansal01yash merged commit e272544 into main Apr 20, 2026
1 check passed
@bansal01yash
bansal01yash deleted the CLOUDGA-33268-cli-role-info branch April 20, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants