eoinmcdonnell113 opened a new pull request, #1142:
URL: https://github.com/apache/ranger/pull/1142

   ## What changes were proposed in this pull request?
   
   RANGER-5730: Restrict createNonExistUserGroupRole during role import so it 
creates missing nested roles only, and does not create missing users or groups.
   
   Previously, importRolesFromFile with createNonExistUserGroupRole=true 
treated users, groups, and nested roles the same way. That forced creation of 
missing users/groups during role import, which is undesirable for large synced 
environments.
   
   This change:
   
   Splits create-missing behavior in RoleRefUpdater into separate controls for 
users/groups vs nested roles
   Updates RoleREST.importRolesFromFile so createNonExistUserGroupRole=true 
means create missing nested roles only (createNonExistUserGroup=false)
   Leaves existing create/update role API behavior (createNonExistUserGroup) 
unchanged
   
   
   ## How was this patch tested?
   
   Unit tests:
   
   TestRoleRefUpdater — verifies missing users are not created when only role 
creation is enabled, and missing nested roles are scheduled for creation
   TestRoleREST — verifies import passes (createNonExistUserGroup=false, 
createNonExistRole=true) when the import flag is true
   TestRoleDBStore — existing create/update role coverage
   Focused run result: TestRoleRefUpdater, TestRoleDBStore, and TestRoleREST — 
102 tests passed (0 failures)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to