Explain why we skip system groups

databrickslabs · Oct 17, 2024 · 65e936e · 65e936e
1 parent 486b6df
commit 65e936e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/databricks/labs/ucx/hive_metastore/grants.py b/src/databricks/labs/ucx/hive_metastore/grants.py
@@ -592,7 +592,7 @@ def _get_external_locations(
 
 
 class PrincipalACL:
-    _system_groups = {"admins", "users"}
+    _workspace_system_groups = {"admins", "users"}
 
     def __init__(
         self,
@@ -694,7 +694,7 @@ def _get_cluster_principal_mapping(self, cluster_id: str, object_type: str) -> l
             if acl.user_name is not None:
                 principal_list.append(acl.user_name)
             if acl.group_name is not None:
-                if acl.group_name in self._system_groups:
+                if acl.group_name in self._workspace_system_groups:  # Do not exists on account level
                     continue
                 principal_list.append(acl.group_name)
             if acl.service_principal_name is not None: