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


Ship it!




Ship It!

- Oliver Szabo


On Jan. 13, 2017, 10:52 a.m., Miklos Gergely wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55492/
> -----------------------------------------------------------
> 
> (Updated Jan. 13, 2017, 10:52 a.m.)
> 
> 
> Review request for Ambari, Oliver Szabo and Robert Nettleton.
> 
> 
> Bugs: AMBARI-19519
>     https://issues.apache.org/jira/browse/AMBARI-19519
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Log Feeder should not put the passwords of the keystore/truststore in the 
> command line, as it may be visible to others. Instead it should be in a file, 
> just like for Log Search Portal.
> 
> 
> Diffs
> -----
> 
>   
> ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/LogFeeder.java
>  6d0f22c 
>   
> ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/util/SSLUtil.java
>  PRE-CREATION 
>   ambari-logsearch/ambari-logsearch-logfeeder/src/main/scripts/run.sh 5aecd00 
>   
> ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py
>  5ffd5e6 
>   
> ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logfeeder.py
>  14f8d20 
>   
> ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py
>  9ff9c74 
>   
> ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logfeeder-env.sh.j2
>  6795dab 
>   ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_logfeeder.py 
> 02570e2 
>   ambari-server/src/test/python/stacks/2.4/configs/default.json 30e12e9 
> 
> Diff: https://reviews.apache.org/r/55492/diff/
> 
> 
> Testing
> -------
> 
> Tested on local cluster
> 
> 
> Thanks,
> 
> Miklos Gergely
> 
>

Reply via email to