Skip to content
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.

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

Migrate Groups - Edge Case - Handle Duplicate Group Name in Account Console #344

Closed
Tracked by #891
nfx opened this issue Sep 29, 2023 · 2 comments
Closed
Tracked by #891
Labels
feat/account-level cross-workspace installations feat/installer install/upgrade the app migrate/groups Corresponds to Migrate Groups Step of go/uc/upgrade needs-triage

Comments

@nfx
Copy link
Collaborator

nfx commented Sep 29, 2023

Background

Within the context of a workspace->account level group migration, we need to handle the following scenario:

  • Workspace Local SCIM is not configured
  • A given workspace local group is an internal, not external group
  • This workspace local group exists in the account console already

Recommended Solution

Create a new group in the account console

  • Prefix group name with the workspace name as follows. workspaceName_groupName
  • Make the workspace admin(s) the group managers for the group
@nfx nfx added enhancement New feature or request feat/account-level cross-workspace installations migrate/groups Corresponds to Migrate Groups Step of go/uc/upgrade feat/installer install/upgrade the app labels Sep 29, 2023
@zpappa zpappa changed the title Create a new group with the workspace name as a prefix, and create in the account console Migrate Groups - Edge Case - Handle Duplicate Group Name in Account Console Sep 29, 2023
@nfx nfx added this to UCX Oct 3, 2023
@nfx nfx moved this to Refined in UCX Oct 3, 2023
@zpappa zpappa moved this from Refined to Triage in UCX Oct 16, 2023
@FastLee
Copy link
Contributor

FastLee commented Nov 27, 2023

A customer has suggested to allow adding a prefix to group names.

@nfx nfx moved this from Triage to Quarter Backlog in UCX Dec 6, 2023
@nfx nfx removed the enhancement New feature or request label Apr 22, 2024
@nfx
Copy link
Collaborator Author

nfx commented Jul 3, 2024

this is already fixed:

validate-groups-membership command
$ databricks labs ucx validate-groups-membership
...
14:30:36 INFO [d.l.u.workspace_access.groups] Found 483 account groups
14:30:36 INFO [d.l.u.workspace_access.groups] No group listing provided, all matching groups will be migrated
14:30:36 INFO [d.l.u.workspace_access.groups] There are no groups with different membership between account and workspace
Workspace Group Name Members Count Account Group Name Members Count Difference
This command validates the groups to see if the groups at the account level and workspace level have different membership. This command is useful for administrators who want to ensure that the groups have the correct membership. It can also be used to debug issues related to group membership. See group migration and group migration for more details.

Valid group membership is important to ensure users has correct access after legacy table ACL is migrated in table migration workflow

@nfx nfx closed this as completed Jul 3, 2024
@github-project-automation github-project-automation bot moved this from Quarter Backlog to Archive in UCX Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat/account-level cross-workspace installations feat/installer install/upgrade the app migrate/groups Corresponds to Migrate Groups Step of go/uc/upgrade needs-triage
Projects
Archived in project
Development

No branches or pull requests

2 participants