You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Sailaja Polavarapu <sp...@hortonworks.com> on 2021/05/14 01:17:51 UTC

Review Request 73355: RANGER-3277: Number of users/groups marked for delete are not shown in logs

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73355/
-----------------------------------------------------------

Review request for ranger, Abhay Kulkarni, Mehul Parikh, Nitin Galave, Ramesh Mani, and Velmurugan Periasamy.


Bugs: RANGER-3277
    https://issues.apache.org/jira/browse/RANGER-3277


Repository: ranger


Description
-------

Added code to log the no. of users/groups that are marked for delete and update it as part of ugsync audits.


Diffs
-----

  security-admin/src/main/java/org/apache/ranger/view/VXFileSyncSourceInfo.java b078a19e3 
  security-admin/src/main/java/org/apache/ranger/view/VXLdapSyncSourceInfo.java be391de71 
  security-admin/src/main/java/org/apache/ranger/view/VXUnixSyncSourceInfo.java 55afec77a 
  security-admin/src/main/webapp/scripts/modules/globalize/message/en.js cc18902cd 
  ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/FileSyncSourceInfo.java d6f12f161 
  ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/LdapSyncSourceInfo.java 2de4fafb8 
  ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/UnixSyncSourceInfo.java 144565588 
  ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java 7e9a364ad 


Diff: https://reviews.apache.org/r/73355/diff/1/


Testing
-------

1. Verified all the existing unit test cases are successful for regressions.
2. Also patched a cluster and tested with unix & AD/LDAP sync sources with deleted users & groups.


Thanks,

Sailaja Polavarapu


Re: Review Request 73355: RANGER-3277: Number of users/groups marked for delete are not shown in logs

Posted by Abhay Kulkarni <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73355/#review223042
-----------------------------------------------------------


Ship it!




Ship It!

- Abhay Kulkarni


On May 14, 2021, 1:17 a.m., Sailaja Polavarapu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73355/
> -----------------------------------------------------------
> 
> (Updated May 14, 2021, 1:17 a.m.)
> 
> 
> Review request for ranger, Abhay Kulkarni, Mehul Parikh, Nitin Galave, Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3277
>     https://issues.apache.org/jira/browse/RANGER-3277
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Added code to log the no. of users/groups that are marked for delete and update it as part of ugsync audits.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/view/VXFileSyncSourceInfo.java b078a19e3 
>   security-admin/src/main/java/org/apache/ranger/view/VXLdapSyncSourceInfo.java be391de71 
>   security-admin/src/main/java/org/apache/ranger/view/VXUnixSyncSourceInfo.java 55afec77a 
>   security-admin/src/main/webapp/scripts/modules/globalize/message/en.js cc18902cd 
>   ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/FileSyncSourceInfo.java d6f12f161 
>   ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/LdapSyncSourceInfo.java 2de4fafb8 
>   ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/UnixSyncSourceInfo.java 144565588 
>   ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java 7e9a364ad 
> 
> 
> Diff: https://reviews.apache.org/r/73355/diff/1/
> 
> 
> Testing
> -------
> 
> 1. Verified all the existing unit test cases are successful for regressions.
> 2. Also patched a cluster and tested with unix & AD/LDAP sync sources with deleted users & groups.
> 
> 
> Thanks,
> 
> Sailaja Polavarapu
> 
>