----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29185/#review65435 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Dec. 18, 2014, 1:32 a.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29185/ > ----------------------------------------------------------- > > (Updated Dec. 18, 2014, 1:32 a.m.) > > > Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. > > > Bugs: AMBARI-8777 > https://issues.apache.org/jira/browse/AMBARI-8777 > > > Repository: ambari > > > Description > ------- > > Added restart hook for clients that will switch the version. NOTE: This is > not all clients, just the "Clients Group" indicated in the runbook. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/script/script.py > b66110d > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > 71bb22c > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java > b450eeb > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java > da784c3 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ColocatedGrouping.java > 0e25c70 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Grouping.java > d68754d > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapperBuilder.java > 0015824 > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_client.py > 043ad11 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_client.py > b9f244a > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_client.py > 499f632 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params.py > bb7f1f4 > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/params.py > 57beac1 > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/pig_client.py > 931dceb > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/mapreduce2_client.py > 831e5e8 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/yarn_client.py > f6b4b44 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/TEZ/package/scripts/params.py > fdf0b21 > > ambari-server/src/main/resources/stacks/HDP/2.1/services/TEZ/package/scripts/tez_client.py > 6ba5ad0 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml > b86aa45 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java > 791c2eb > ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_client.py > 9124d64 > ambari-server/src/test/python/stacks/2.0.6/HDFS/test_hdfs_client.py be0fac1 > ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py > 0c793c8 > ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_client.py 39e5b8d > ambari-server/src/test/python/stacks/2.0.6/configs/client-upgrade.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.1/TEZ/test_tez_client.py 7b71d56 > ambari-server/src/test/python/stacks/2.1/configs/client-upgrade.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/29185/diff/ > > > Testing > ------- > > Added tests for most of the clients, the ones that I had installed at the > time. Can verify the others when those services become completed (read: > HIVE). > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 30:01.173s > [INFO] Finished at: Wed Dec 17 20:21:53 EST 2014 > [INFO] Final Memory: 31M/340M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Nate Cole > >
