ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Velmurugan Periasamy <vperias...@hortonworks.com>
Subject Re: Review Request 65871: RANGER-2004: updated Ranger authorization plugin for Atlas for the changes in ATLAS-2459
Date Sat, 03 Mar 2018 03:23:10 GMT


> On March 1, 2018, 11:16 p.m., Velmurugan Periasamy wrote:
> > agents-common/src/main/resources/service-defs/ranger-servicedef-atlas.json
> > Lines 12 (patched)
> > <https://reviews.apache.org/r/65871/diff/1/?file=1968607#file1968607line12>
> >
> >     Madhan - I believe existing atlas service definition needs to be updated during
upgrades, right?
> 
> Madhan Neethiraj wrote:
>     Vel - given the scope of the authorization model changes, current service-def can't
be migrated to the new model. We will have to rename it (better delete it) and register the
new service-def as 'atlas'

Would there be a Java patch to address this?


- Velmurugan


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


On March 2, 2018, 8:48 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65871/
> -----------------------------------------------------------
> 
> (Updated March 2, 2018, 8:48 a.m.)
> 
> 
> Review request for ranger, Gautam Borad, Abhay Kulkarni, Mehul Parikh, Nixon Rodrigues,
Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2004
>     https://issues.apache.org/jira/browse/RANGER-2004
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Apache Atlas updated the authorization model in ATLAS-2459, to support authroization
at instance level. Ranger authorization plugin for Atlas has been updated to support this
model.
> (publishing review for the patch from Nixon)
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/resources/service-defs/ranger-servicedef-atlas.json 4a550c64

>   plugin-atlas/src/main/java/org/apache/ranger/authorization/atlas/authorizer/RangerAtlasAuthorizer.java
90e75a1a 
>   plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
fe978744 
>   plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasClient.java
ea05ad0f 
>   plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasConnectionMgr.java
140f91e8 
>   plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java
f81e3041 
> 
> 
> Diff: https://reviews.apache.org/r/65871/diff/3/
> 
> 
> Testing
> -------
> 
> Deployed the updated plugin and verified authorization and audit logs for various scenarios
(entity-read, entity-add-classification, entity-remove-classification, ..).
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message