----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62402/#review185670 -----------------------------------------------------------
Ship it! Ship It! - Attila Doroszlai On Sept. 19, 2017, 10:53 a.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62402/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2017, 10:53 a.m.) > > > Review request for Ambari, Myroslav Papirkovskyy, Sumit Mohanty, and Vitalyi > Brodetskyi. > > > Bugs: AMBARI-21989 > https://issues.apache.org/jira/browse/AMBARI-21989 > > > Repository: ambari > > > Description > ------- > > * restart triggered by server should gracefully finish operations > * if status command fails, exception should be logged. > * move configs logic from InitializerModule into AmbariConfig, where it > belongs > * fix typo (alert_def*E*nitions into alert_def*I*nitions) > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/AlertSchedulerHandler.py 94e72c2 > ambari-agent/src/main/python/ambari_agent/AlertStatusReporter.py 2bd2383 > ambari-agent/src/main/python/ambari_agent/AmbariConfig.py e1c40c8 > ambari-agent/src/main/python/ambari_agent/CommandStatusDict.py c681550 > ambari-agent/src/main/python/ambari_agent/CommandStatusReporter.py 652574f > ambari-agent/src/main/python/ambari_agent/ComponentStatusExecutor.py > f85ba42 > ambari-agent/src/main/python/ambari_agent/Constants.py 6a258d2 > ambari-agent/src/main/python/ambari_agent/FileCache.py 0e598e9 > ambari-agent/src/main/python/ambari_agent/HeartbeatThread.py b6e1aaf > ambari-agent/src/main/python/ambari_agent/HostStatusReporter.py 09ce016 > ambari-agent/src/main/python/ambari_agent/InitializerModule.py 0126250 > ambari-agent/src/main/python/ambari_agent/Utils.py af70d77 > > ambari-agent/src/main/python/ambari_agent/listeners/AlertDefinitionsEventListener.py > 91ae5de > ambari-agent/src/main/python/ambari_agent/main.py 05fc5ce > ambari-agent/src/test/python/ambari_agent/TestAgentStompResponses.py > 3c571ab > > > Diff: https://reviews.apache.org/r/62402/diff/1/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >