> On June 2, 2016, 4:48 a.m., Velmurugan Periasamy wrote:
> > security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml, 
> > line 462
> > <https://reviews.apache.org/r/48110/diff/1/?file=1402982#file1402982line462>
> >
> >     In case of Ambari based install, are there any changes required?
> 
> Ankita Sinha wrote:
>     Yes we need Ambari based changes and it's already committed

thanks. then this is good to go.


- Velmurugan


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


On June 1, 2016, 5:26 a.m., Ankita Sinha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48110/
> -----------------------------------------------------------
> 
> (Updated June 1, 2016, 5:26 a.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, 
> Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, and 
> Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1010
>     https://issues.apache.org/jira/browse/RANGER-1010
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Make "ranger.sso.cookiename" and "ranger.sso.query.param.originalurl" 
> properties non-configurable, add them to ranger-admin-default-site.xml and 
> remove them from ranger-admin-site.xml and also remove them from 
> install.properties.
> 
> 
> Diffs
> -----
> 
>   kms/config/kms-webapp/dbks-site.xml a82a72b 
>   security-admin/scripts/install.properties d2cbf45 
>   security-admin/scripts/setup.sh c6defd8 
>   security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml 
> aee1c82 
>   security-admin/src/main/resources/conf.dist/ranger-admin-site.xml 60a2c96 
> 
> Diff: https://reviews.apache.org/r/48110/diff/
> 
> 
> Testing
> -------
> 
> 1. Ranger Admin with Knox SSO enable
> 2. Ranger Admin with Knox SSO disable
> 
> 
> Thanks,
> 
> Ankita Sinha
> 
>

Reply via email to