----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35641/#review88485 -----------------------------------------------------------
Ship it! ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py (line 659) <https://reviews.apache.org/r/35641/#comment141054> Should we try to be consistent in the hosts we use in tests? Maybe c6401.ambari.apache.org ? - Jonathan Hurley On June 18, 2015, 10:55 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35641/ > ----------------------------------------------------------- > > (Updated June 18, 2015, 10:55 p.m.) > > > Review request for Ambari and Jonathan Hurley. > > > Repository: ambari > > > Description > ------- > > There's a known issue in HDP 2.2.0.0 (HDFS-7533) where shutting down the > datanode will not work because not all writers have responder running, but > sendOOB() tries anyway. > If the shutdown command fails with an output of "Shutdown already in > progress", then Ambari should call datanode(action="stop"), which under the > hood calls "hadoop-daemon.sh stop datanode" > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/datanode.py > d8b8835 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/datanode_upgrade.py > 29af5bd > ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 2440145 > ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py > da85cfc > > Diff: https://reviews.apache.org/r/35641/diff/ > > > Testing > ------- > > Uploaded the patch to a cluster performing a Rolling Upgrade and verified it > worked. > > Unit tests passed. > > ---------------------------------------------------------------------- > Total run:762 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >
