Skip to content

chore(UPM-61578): Fixed HM role names #7030

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mpfuster merged 11 commits into from
Dec 29, 2025
Merged

chore(UPM-61578): Fixed HM role names #7030

mpfuster merged 11 commits into from
Dec 29, 2025

Conversation

@crudo
Copy link

@crudo crudo commented Dec 3, 2025 •
edited
Loading

This pull request updates the documentation for user roles in Hybrid Manager to standardize and clarify the naming of roles. The changes improve consistency and readability by using title case for all role names and aligning them with their functional descriptions.

Documentation updates for user roles:

  • Standardized all role names to title case (e.g., "Organization administrator" → "Organization Administrator", "Project owner" → "Project Owner") for consistency and clarity.
  • Updated some role names to better reflect their functions, such as changing "GenAI Builder user" to "GenAI Builder Editor".
  • Applied these naming conventions to both organization-level and project-level roles throughout the documentation.

See https://github.com/EnterpriseDB/upm-beaco-ff-base/pull/443

Role ID Old name New name
rol_org_appliance_admin platform admin Platform Administrator
rol_org_owner owner Organization Owner
rol_org_admin admin Organization Administrator
rol_prj_catalog_data_reader catalog data reader Catalog Data Reader
rol_prj_catalog_data_writer catalog data writer Catalog Data Writer
rol_prj_migration_portal_projects_editor Migration Portal projects editor Migration Portal Projects Editor
rol_prj_migration_portal_projects_owner Migration Portal projects owner Migration Portal Projects Owner
rol_prj_migration_portal_projects_viewer Migration Portal projects viewer Migration Portal Projects Viewer
rol_prj_owner owner Project Owner
rol_prj_editor editor Project Editor
rol_prj_viewer viewer Project Viewer
rol_prj_pipeline_editor Pipeline editor Pipeline Editor
rol_org_genai_builder_user GenAI Builder user GenAI Builder Editor
rol_prj_estate_ingester estate ingester Estate Ingester

@crudo crudo requested a review from a team as a code owner December 3, 2025 11:48
ebgitelman
ebgitelman reviewed Dec 12, 2025
View reviewed changes
...ostgres-ai/1.2/hybrid-manager/using_hybrid_manager/managing_users/user_roles_authz/roles.mdx Outdated
Users with this role can read the Iceberg namespace/table/view of project-scoped catalogs.

### Catalog data writer (catalog data writer)
### Catalog Data data writer (catalog data writer)
Copy link
Contributor

@ebgitelman ebgitelman Dec 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this correct?

Copy link
Contributor

@xugy99 xugy99 Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, the change only applies to HM preview version.

ebgitelman
ebgitelman reviewed Dec 12, 2025
View reviewed changes
...ostgres-ai/1.2/hybrid-manager/using_hybrid_manager/managing_users/user_roles_authz/roles.mdx Outdated
This role is for machine users and provides access to perform estate ingests in a project.

### Catalog data reader (catalog data reader)
### Catalog Data data reader (catalog data reader)
Copy link
Contributor

@ebgitelman ebgitelman Dec 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct?

Copy link
Contributor

@xugy99 xugy99 Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change only applies to preview, not 1.2.

ebgitelman
ebgitelman reviewed Dec 12, 2025
View reviewed changes
Copy link
Contributor

@ebgitelman ebgitelman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Puzzled that the names of roles weren't changed in the 1.2 doc?

@mpfuster mpfuster added the deploy Add this label to a PR and it will automatically be deployed to netlify label Dec 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025 •
edited
Loading

🚀 Deployed on https://deploy-preview-7030--edb-docs-staging.netlify.app

@github-actions github-actions bot temporarily deployed to pull request December 15, 2025 09:44 Inactive
@irmabenitezzapata-creator
Copy link
Contributor

irmabenitezzapata-creator commented Dec 15, 2025

Hi @crudo, @xugy99 do these changes also apply for 1.3?

xugy99 and others added 7 commits December 24, 2025 15:01
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/preview/hybrid-manager/using…
f419bf8 
…_hybrid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/preview/hybrid-manager/using…
2d55031 
…_hybrid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/preview/hybrid-manager/using…
36761b1 
…_hybrid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/preview/hybrid-manager/using…
30338d8 
…_hybrid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/preview/hybrid-manager/using…
14a10b2 
…_hybrid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/preview/hybrid-manager/using…
69e945b 
…_hybrid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@xugy99 @ebgitelman
Update product_docs/docs/edb-postgres-ai/1.2/hybrid-manager/using_hyb…
dfa4ce7 
…rid_manager/managing_users/user_roles_authz/roles.mdx

Co-authored-by: Betsy Gitelman <[email protected]>
@github-actions github-actions bot temporarily deployed to pull request December 24, 2025 07:33 Inactive
@ebgitelman
Apply suggestions from code review
5d05865 
Think this should be good to go now.
@github-actions github-actions bot temporarily deployed to pull request December 26, 2025 17:48 Inactive
mpfuster
mpfuster approved these changes Dec 29, 2025
View reviewed changes
Copy link
Contributor

@mpfuster mpfuster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mpfuster mpfuster merged commit 14f7251 into develop Dec 29, 2025
6 checks passed
@mpfuster mpfuster deleted the c-upm-61578 branch December 29, 2025 09:21
@github-actions github-actions bot temporarily deployed to pull request December 29, 2025 09:24 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xugy99 xugy99 xugy99 left review comments

@ebgitelman ebgitelman ebgitelman left review comments

@mpfuster mpfuster mpfuster approved these changes

Assignees

No one assigned

Labels

deploy Add this label to a PR and it will automatically be deployed to netlify

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@crudo @irmabenitezzapata-creator @xugy99 @ebgitelman @mpfuster