----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29313/#review65790 -----------------------------------------------------------
Ship it! Thanks for submitting this. - Alejandro Fernandez On Dec. 22, 2014, 4:54 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29313/ > ----------------------------------------------------------- > > (Updated Dec. 22, 2014, 4:54 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Mahadev > Konar. > > > Bugs: AMBARI-8861 > https://issues.apache.org/jira/browse/AMBARI-8861 > > > Repository: ambari > > > Description > ------- > > Zookeeper scripts are only checking for ZOOKEEPER, ZOOKEEPER_CLIENT for > determining the correct path. > > Also fixed a typo that's been annoying me for months :) > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py > 61997f2 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/params.py > 929d5d8 > > ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_service_check.py > ade731f > > ambari-server/src/test/python/stacks/2.0.6/configs/zk-service_check_2.2.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/29313/diff/ > > > Testing > ------- > > Added python test for ZK service check when using 2.2+ > > ---------------------------------------------------------------------- > Total run:680 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Nate Cole > >
