----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46927/#review131767 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/pacemaker.py (line 37) <https://reviews.apache.org/r/46927/#comment195806> Is Pacemaker a new component? I dont see any change in metainfo to add this as a new component. ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/pacemaker.py (line 40) <https://reviews.apache.org/r/46927/#comment195805> Shouldnt return "storm-client" - Jayush Luniya On May 3, 2016, 4:52 a.m., Sriharsha Chintalapani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46927/ > ----------------------------------------------------------- > > (Updated May 3, 2016, 4:52 a.m.) > > > Review request for Ambari, Jaimin Jetly and Sumit Mohanty. > > > Repository: ambari > > > Description > ------- > > AMBARI-16210. Update Storm Ambari stack definition to work with 1.0 release. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/configuration/storm-site.xml > 5a3901a > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/pacemaker.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/service.py > d0b717f > > ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/configuration/storm-site.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/kerberos.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/metainfo.xml > a296bfa > > Diff: https://reviews.apache.org/r/46927/diff/ > > > Testing > ------- > > Deployed a 3 node cluster and ran sample topologies. Also kerbeorized the > cluster. > > > Thanks, > > Sriharsha Chintalapani > >