You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2020/04/06 06:53:35 UTC

Review Request 72320: RANGER-2780: updated RangerBasePlugin to support audit-excluded-users

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

Review request for ranger, deepak sharma, Kishor Gollapalliwar, Abhay Kulkarni, Mehul Parikh, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy.


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


Repository: ranger


Description
-------

- added method RangerBasePlugin.setAuditExcludedUsersGroupsRoles(), which a plugin implementation can call to set users/groups/roles for whom audit logs should be skipped


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerPluginConfig.java cea6842cf 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/PolicyEngine.java bc8067789 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngine.java c6731696a 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java c68e22df3 
  agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java b5e18ba64 
  agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyEngine.java 8811f2ad0 
  agents-common/src/test/resources/policyengine/test_policyengine_audit_exclude_users_groups_roles.json PRE-CREATION 


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


Testing
-------

- added unit tests to cover the enhancements
- verified that existing tests pass successfully


Thanks,

Madhan Neethiraj


Re: Review Request 72320: RANGER-2780: updated RangerBasePlugin to support audit-excluded-users

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


Ship it!




Ship It!

- Abhay Kulkarni


On April 6, 2020, 6:53 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72320/
> -----------------------------------------------------------
> 
> (Updated April 6, 2020, 6:53 a.m.)
> 
> 
> Review request for ranger, deepak sharma, Kishor Gollapalliwar, Abhay Kulkarni, Mehul Parikh, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2780
>     https://issues.apache.org/jira/browse/RANGER-2780
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> - added method RangerBasePlugin.setAuditExcludedUsersGroupsRoles(), which a plugin implementation can call to set users/groups/roles for whom audit logs should be skipped
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/authorization/hadoop/config/RangerPluginConfig.java cea6842cf 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/PolicyEngine.java bc8067789 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngine.java c6731696a 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java c68e22df3 
>   agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java b5e18ba64 
>   agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyEngine.java 8811f2ad0 
>   agents-common/src/test/resources/policyengine/test_policyengine_audit_exclude_users_groups_roles.json PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/72320/diff/1/
> 
> 
> Testing
> -------
> 
> - added unit tests to cover the enhancements
> - verified that existing tests pass successfully
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>