----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62388/#review185654 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/package/scripts/service.py Lines 84 (patched) <https://reviews.apache.org/r/62388/#comment261978> We can use checked_call here. So we get more context from exception message if return code is non_zero. In case if it is zero, but regex does not match. We need to also print output in exception text. To give debugging information for failures of ATS. - Andrew Onischuk On Sept. 18, 2017, 6 p.m., Eugene Chekanskiy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62388/ > ----------------------------------------------------------- > > (Updated Sept. 18, 2017, 6 p.m.) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Robert > Levas. > > > Bugs: AMBARI-21984 > https://issues.apache.org/jira/browse/AMBARI-21984 > > > Repository: ambari > > > Description > ------- > > Sometimes scripts starting yarn services but they are already running. It is > rarely reproduced and hard to debug. > This patch checks output of start command and ignore start failure in case > script reporting "service is running" and validates pid pile after that. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service.py > 7c59b60 > > ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/package/scripts/service.py > 1c612ef > ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py > 6a99dfa > ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py 642043d > ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py > 8a703df > ambari-server/src/test/python/stacks/2.1/YARN/test_apptimelineserver.py > b352920 > > > Diff: https://reviews.apache.org/r/62388/diff/1/ > > > Testing > ------- > > mvn clean test, cluster deploy > > > Thanks, > > Eugene Chekanskiy > >
