----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51522/#review147295 -----------------------------------------------------------
ambari-server/conf/unix/ambari.properties (line 127) <https://reviews.apache.org/r/51522/#comment214414> Lets name the file ambari-server-command.log ambari-server/src/main/python/ambari-server.py (line 120) <https://reviews.apache.org/r/51522/#comment214415> Starting ambari-server should we add a line after the server_process_main() call to say "Started ambari-server. ambari-server/src/main/python/ambari-server.py (line 156) <https://reviews.apache.org/r/51522/#comment214416> Stopping ambari-server. Can we also add "Stopped ambari-server" at the end. - Sumit Mohanty On Aug. 30, 2016, 12:27 p.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51522/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2016, 12:27 p.m.) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, > and Sumit Mohanty. > > > Bugs: AMBARI-18273 > https://issues.apache.org/jira/browse/AMBARI-18273 > > > Repository: ambari > > > Description > ------- > > Add ambari server python side logging.. > > > Diffs > ----- > > ambari-server/conf/unix/ambari.properties b8e41b7 > ambari-server/sbin/ambari-server 8064c45 > ambari-server/src/main/python/ambari-server.py f929042 > ambari-server/src/main/python/ambari_server/checkDatabase.py 831efde > ambari-server/src/main/python/ambari_server/dbCleanup.py 7a5486c > ambari-server/src/main/python/ambari_server/hostUpdate.py d5ab22a > ambari-server/src/main/python/ambari_server/serverSetup.py d80c25a > ambari-server/src/main/python/ambari_server/serverUpgrade.py d3cb3ba > ambari-server/src/main/python/ambari_server/setupActions.py 182c11e > ambari-server/src/main/python/ambari_server/setupMpacks.py f854d76 > ambari-server/src/main/python/ambari_server/setupSecurity.py 69579cd > ambari-server/src/main/python/ambari_server/setupSso.py 2a5e44c > ambari-server/src/main/python/ambari_server_main.py 17945de > ambari-server/src/test/python/TestAmbariServer.py a45a4bd > > Diff: https://reviews.apache.org/r/51522/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Vitalyi Brodetskyi > >
