----------------------------------------------------------- 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 > >
