----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71999/#review219362 -----------------------------------------------------------
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPluginInfo.java Line 44 (original), 44 (patched) <https://reviews.apache.org/r/71999/#comment307507> nit: Please remove the white spaces and make as it was before agents-common/src/main/java/org/apache/ranger/plugin/util/RangerUserStoreUtil.java Lines 48 (patched) <https://reviews.apache.org/r/71999/#comment307506> "cloud_id", please make it a constant and refer where ever used. security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Lines 2551 (patched) <https://reviews.apache.org/r/71999/#comment307508> please consider doing return daoManager.getXXGlobalState().getAppDataVersion(RANGER_USER_GROUP_GLOBAL_STATE_NAME); security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Lines 2558 (patched) <https://reviews.apache.org/r/71999/#comment307509> same as above comment on return statement ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapDeltaUserGroupBuilder.java Lines 200 (patched) <https://reviews.apache.org/r/71999/#comment307510> please make "byte[]" a constant and refer to it where ever used. - Ramesh Mani On Jan. 22, 2020, 7:02 a.m., Sailaja Polavarapu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71999/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2020, 7:02 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-2697 > https://issues.apache.org/jira/browse/RANGER-2697 > > > Repository: ranger > > > Description > ------- > > Modified Usersync code to retrieve configurable user and group attributes > from AD/LDAP. Also added rest API support in ranger admin to dowmload user > and group information including these additional attributes. > > > Diffs > ----- > > agents-common/src/main/java/org/apache/ranger/plugin/model/GroupInfo.java > PRE-CREATION > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPluginInfo.java > 90367fe04 > agents-common/src/main/java/org/apache/ranger/plugin/model/UserInfo.java > PRE-CREATION > > agents-common/src/main/java/org/apache/ranger/plugin/util/RangerUserStore.java > PRE-CREATION > > agents-common/src/main/java/org/apache/ranger/plugin/util/RangerUserStoreUtil.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/biz/AssetMgr.java 0f4488861 > security-admin/src/main/java/org/apache/ranger/biz/RoleDBStore.java > 04596dc65 > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > ccda6abb0 > security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java 3045eaf9a > security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 037c591e8 > > security-admin/src/main/java/org/apache/ranger/common/RangerUserStoreCache.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/db/XXGlobalStateDao.java > 2e462bd3a > security-admin/src/main/java/org/apache/ranger/rest/XUserREST.java > 1e8a09379 > > security-admin/src/main/java/org/apache/ranger/service/XGroupServiceBase.java > 1a701bbfb > > security-admin/src/main/java/org/apache/ranger/service/XUserServiceBase.java > 1004952a9 > > ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapDeltaUserGroupBuilder.java > ca50f098a > > ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java > b469e9245 > > ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java > 8efa1613a > > ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java > 1d4e37fcf > ugsync/src/main/java/org/apache/ranger/unixusersync/model/MUserInfo.java > 841bac64a > ugsync/src/main/java/org/apache/ranger/unixusersync/model/XGroupInfo.java > cbe0eb02b > ugsync/src/main/java/org/apache/ranger/unixusersync/model/XUserInfo.java > e2f36b2c5 > > ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java > f08c5117b > ugsync/src/main/java/org/apache/ranger/usergroupsync/UserGroupSink.java > 77bd016fd > > ugsync/src/test/java/org/apache/ranger/usergroupsync/LdapPolicyMgrUserGroupBuilderTest.java > 99bc2b44e > > ugsync/src/test/java/org/apache/ranger/usergroupsync/PolicyMgrUserGroupBuilderTest.java > 2bc395180 > > > Diff: https://reviews.apache.org/r/71999/diff/2/ > > > Testing > ------- > > 1. Verified all the existing unit tests are run successfully. > 2. Patched a cluster and ran tests to sync various user/group attributes from > test AD > 3. Also verified new rest API to download user and group information with > addition attributes. > > > Thanks, > > Sailaja Polavarapu > >