You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Nikhil P <ni...@gmail.com> on 2017/09/08 08:28:17 UTC

Review Request 62078: Update NiFi service def and handle upgrade scenario.

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

Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.


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


Repository: ranger


Description
-------

Apache NiFi would like to update the service def file to ensure more consistent policy support between NiFi and Ranger. NiFi is in the process of introducing an improved Ranger authorizer that allows the policies to be viewed in NiFi UI for easier, in app, understanding of the policies configured in Ranger UI. This effort is being tracked with this JIRA.


Diffs
-----

  agents-common/src/main/resources/service-defs/ranger-servicedef-nifi.json b81785d 
  security-admin/src/main/java/org/apache/ranger/patch/PatchForNifiResourceUpdateExclude_J10008.java PRE-CREATION 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 6de8a14 
  security-admin/src/main/java/org/apache/ranger/service/XTrxLogService.java 6736c56 


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


Testing
-------

1)Tested on fresh install to check that exclude toggle is not visible on create/update policy page.
2)Tested on upgrade case for Mysql to check that exclude toggle is not visible on create/update policy page.
3)In ugrade scenario,checked if existing exclude polcies gets read only after upgrade.


Thanks,

Nikhil P


Re: Review Request 62078: Update NiFi service def and handle upgrade scenario.

Posted by Velmurugan Periasamy <vp...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62078/#review185006
-----------------------------------------------------------


Ship it!




Ship It!

- Velmurugan Periasamy


On Sept. 8, 2017, 8:28 a.m., Nikhil P wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62078/
> -----------------------------------------------------------
> 
> (Updated Sept. 8, 2017, 8:28 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1697
>     https://issues.apache.org/jira/browse/RANGER-1697
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Apache NiFi would like to update the service def file to ensure more consistent policy support between NiFi and Ranger. NiFi is in the process of introducing an improved Ranger authorizer that allows the policies to be viewed in NiFi UI for easier, in app, understanding of the policies configured in Ranger UI. This effort is being tracked with this JIRA.
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/resources/service-defs/ranger-servicedef-nifi.json b81785d 
>   security-admin/src/main/java/org/apache/ranger/patch/PatchForNifiResourceUpdateExclude_J10008.java PRE-CREATION 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 6de8a14 
>   security-admin/src/main/java/org/apache/ranger/service/XTrxLogService.java 6736c56 
> 
> 
> Diff: https://reviews.apache.org/r/62078/diff/1/
> 
> 
> Testing
> -------
> 
> 1)Tested on fresh install to check that exclude toggle is not visible on create/update policy page.
> 2)Tested on upgrade case for Mysql to check that exclude toggle is not visible on create/update policy page.
> 3)In ugrade scenario,checked if existing exclude polcies gets read only after upgrade.
> 
> 
> Thanks,
> 
> Nikhil P
> 
>


Re: Review Request 62078: Update NiFi service def and handle upgrade scenario.

Posted by Gautam Borad <gb...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62078/#review185074
-----------------------------------------------------------


Ship it!




Ship It!

- Gautam Borad


On Sept. 8, 2017, 8:28 a.m., Nikhil P wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62078/
> -----------------------------------------------------------
> 
> (Updated Sept. 8, 2017, 8:28 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1697
>     https://issues.apache.org/jira/browse/RANGER-1697
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Apache NiFi would like to update the service def file to ensure more consistent policy support between NiFi and Ranger. NiFi is in the process of introducing an improved Ranger authorizer that allows the policies to be viewed in NiFi UI for easier, in app, understanding of the policies configured in Ranger UI. This effort is being tracked with this JIRA.
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/resources/service-defs/ranger-servicedef-nifi.json b81785d 
>   security-admin/src/main/java/org/apache/ranger/patch/PatchForNifiResourceUpdateExclude_J10008.java PRE-CREATION 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 6de8a14 
>   security-admin/src/main/java/org/apache/ranger/service/XTrxLogService.java 6736c56 
> 
> 
> Diff: https://reviews.apache.org/r/62078/diff/1/
> 
> 
> Testing
> -------
> 
> 1)Tested on fresh install to check that exclude toggle is not visible on create/update policy page.
> 2)Tested on upgrade case for Mysql to check that exclude toggle is not visible on create/update policy page.
> 3)In ugrade scenario,checked if existing exclude polcies gets read only after upgrade.
> 
> 
> Thanks,
> 
> Nikhil P
> 
>