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


Ship it!




Ship It!

- Nikhil P


On June 11, 2020, 5:55 p.m., Pradeep Agrawal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72585/
> -----------------------------------------------------------
> 
> (Updated June 11, 2020, 5:55 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Don Bosco Durai, bhavik patel, Colm 
> O hEigeartaigh, Gautam Borad, Jayendra Parab, Abhay Kulkarni, Madhan 
> Neethiraj, Mehul Parikh, Nikhil P, Nitin Galave, pengjianhua, Ramesh Mani, 
> Selvamohan Neethiraj, Sailaja Polavarapu, Velmurugan Periasamy, Qiang Zhang, 
> and Zsombor Gegesy.
> 
> 
> Bugs: RANGER-2854
>     https://issues.apache.org/jira/browse/RANGER-2854
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> **Problem Statement:** Current bootstrap properties are not available in 
> install.properties so at the first time installation user have to manually 
> change in the ranger-admin-site.xml. Also after the elastic search audit 
> feature these property should be honoured based on ranger audit source type 
> which can be solr or elastic search.
> 
> **Proposed solution:** by Exposing bootstrap configs in install.properties, 
> setup.sh shall update them in ranger-admin-site.xml. During the start of 
> embeddedserver based on ranger audit source type relevant bootstrap property 
> will be referred. By default bootstrap config values are true.
> 
> 
> Diffs
> -----
> 
>   
> embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java
>  dc931c25d 
>   security-admin/scripts/install.properties cb79d1295 
>   security-admin/scripts/setup.sh 61d351dde 
>   security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml 
> 3eb47f932 
>   security-admin/src/main/resources/conf.dist/ranger-admin-site.xml 5dc14ab21 
> 
> 
> Diff: https://reviews.apache.org/r/72585/diff/1/
> 
> 
> Testing
> -------
> 
> Tested Ranger installation with true and false values; values are getting 
> populated in ranger-admin-site.xml file.
> 
> 
> Thanks,
> 
> Pradeep Agrawal
> 
>

Reply via email to