----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55119/#review160400 -----------------------------------------------------------
Ship it! Ship It! - Robert Levas On Jan. 3, 2017, 10:50 a.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55119/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2017, 10:50 a.m.) > > > Review request for Ambari, Miklos Gergely, Robert Levas, and Robert Nettleton. > > > Bugs: AMBARI-19333 > https://issues.apache.org/jira/browse/AMBARI-19333 > > > Repository: ambari > > > Description > ------- > > Strore logsearch keystore/truststore password in a file. (istead of using it > as an environment variable in the start script) > file persmissions for passwords are 600. so not encrypted (maybe it can be in > the future, but not urgent) > > The change does not contain any ambari credential store solution. (that is > something on ambari side, this solution is mainly for logsearch side) > > > Diffs > ----- > > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/LogSearch.java > 614e91e > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/util/SSLUtil.java > 7a93305 > ambari-logsearch/ambari-logsearch-portal/src/main/scripts/run.sh 1204ef3 > ambari-logsearch/docker/test-config/logsearch/logsearch-env.sh 2c2d056 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py > 4d751ba > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py > 874b90b > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logsearch-env.sh.j2 > a179983 > ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_logsearch.py > c3e8930 > ambari-server/src/test/python/stacks/2.4/configs/default.json 7591adb > > Diff: https://reviews.apache.org/r/55119/diff/ > > > Testing > ------- > > testing done > > > Thanks, > > Oliver Szabo > >