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