----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54189/#review157356 -----------------------------------------------------------
Ship it! Ship It! - Swapan Shridhar On Nov. 30, 2016, 1:15 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54189/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2016, 1:15 a.m.) > > > Review request for Ambari, Mugdha Varadkar, Sumit Mohanty, and Swapan > Shridhar. > > > Bugs: AMBARI-19023 > https://issues.apache.org/jira/browse/AMBARI-19023 > > > Repository: ambari > > > Description > ------- > > Whenever adding an Atlas Server on a host, if the user executes the first > command as a "restart" (stop+start) instead of a "start" command, then the > stop command will fail because Ambari will not have generated any configs yet > so Atlas' atlas_stop.py will fail. > > To fix it, during a stop command, > If the pid dir doesn't exist, this means either > 1. The user just added Atlas service and issued a restart command > (stop+start). So stop should be a no-op since there's nothing to stop because > the pid dir does not exist. > > OR > > 2. The user changed the value of the pid dir config and incorrectly issued a > restart command. > In which case the stop command cannot do anything since Ambari doesn't know > which process to kill. > The start command will spawn another instance. > The user should have issued a stop, changed the config, and then started it. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py > a469ebb > ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py > 585dc94 > > Diff: https://reviews.apache.org/r/54189/diff/ > > > Testing > ------- > > Verified on a life cluster. > > Python unit tests passed. > ---------------------------------------------------------------------- > Total run:1166 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >