> On June 21, 2016, 4:56 p.m., Sebastian Toader wrote:
> > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch_solr.py,
> >  lines 96-100
> > <https://reviews.apache.org/r/49033/diff/1/?file=1426743#file1426743line96>
> >
> >     Instead of having two Execute commands have just only one and set the 
> > value for security_content properly.

unfortunately it expects python variable inside {}, so in the end it looks more 
readable (if im not put into format)..and if i add escaping the python string 
will recognize that.


- Oliver


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


On June 21, 2016, 4:52 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49033/
> -----------------------------------------------------------
> 
> (Updated June 21, 2016, 4:52 p.m.)
> 
> 
> Review request for Ambari, Don Bosco Durai, Miklos Gergely, Robert Nettleton, 
> Sumit Mohanty, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-17344
>     https://issues.apache.org/jira/browse/AMBARI-17344
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> /security.json needs to be cleared if kerberos is disabled. can be a problem 
> if a cluster is kerberized, then kerberos is disabled after that
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch_solr.py
>  33ba70c 
>   ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_solr.py 0474aed 
> 
> Diff: https://reviews.apache.org/r/49033/diff/
> 
> 
> Testing
> -------
> 
> Total run:1073
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>

Reply via email to