----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29778/#review67534 -----------------------------------------------------------
Ship it! ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml <https://reviews.apache.org/r/29778/#comment111571> Storm should be marked skippable. - Nate Cole On Jan. 9, 2015, 5:08 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29778/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2015, 5:08 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Nate Cole. > > > Bugs: AMBARI-9067 > https://issues.apache.org/jira/browse/AMBARI-9067 > > > Repository: ambari > > > Description > ------- > > Storm upgrade will include a manual pre-upgrade task > {{Deactivate and kill your currently running topology.}} > and a manual post-upgrade task > {{Rebuild your topology and resubmit the topology using the new jar.}} > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/drpc_server.py > 149b212 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/nimbus.py > 6d68f3e > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/nimbus_prod.py > a1f8afe > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/params.py > 6d49d04 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/rest_api.py > 890c329 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/supervisor.py > 55f191e > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/supervisor_prod.py > 7aecdc5 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/ui_server.py > 6a9d923 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml > ed6b50c > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_drpc_server.py > 336aded > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_nimbus.py d9887d4 > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_nimbus_prod.py > 6c7ea30 > > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_rest_api_service.py > 4e29ffa > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_supervisor.py > 699dce8 > > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_supervisor_prod.py > 4ae0c4d > ambari-server/src/test/python/stacks/2.1/STORM/test_storm_ui_server.py > becd591 > ambari-server/src/test/python/stacks/2.1/configs/default.json 9f22fe8 > > Diff: https://reviews.apache.org/r/29778/diff/ > > > Testing > ------- > > New unit tests added. All tests pass. > > > Thanks, > > Tom Beerbower > >
