> On June 19, 2015, 3:25 a.m., Jonathan Hurley wrote: > > ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py, line > > 659 > > <https://reviews.apache.org/r/35641/diff/1/?file=987978#file987978line659> > > > > Should we try to be consistent in the hosts we use in tests? Maybe > > c6401.ambari.apache.org ?
This was a unit test that was failing. I'm not sure where the HW11079 value is coming from since I found no other occurrences of it when I grepped for it. - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35641/#review88485 ----------------------------------------------------------- On June 19, 2015, 2:55 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35641/ > ----------------------------------------------------------- > > (Updated June 19, 2015, 2:55 a.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 > >
