----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26511/#review56095 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Oct. 9, 2014, 7:58 p.m., Sumit Mohanty wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26511/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2014, 7:58 p.m.) > > > Review request for Ambari, Srimanth Gunturi and Sid Wagle. > > > Bugs: AMBARI-7713 > https://issues.apache.org/jira/browse/AMBARI-7713 > > > Repository: ambari > > > Description > ------- > > Slider service should specify what client configs can be downloaded. Some > minor changes such as dropping property "slider.yarn.queue" and moving the > slider-env.sh to a configurable template. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/configuration/slider-client.xml > 7d55760 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/configuration/slider-env.xml > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/metainfo.xml > 946fad8 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/package/scripts/params.py > 3b12f9b > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/package/scripts/slider.py > a3b6238 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/package/templates/slider-env.sh.j2 > 4872e29 > ambari-server/src/test/python/stacks/2.2/SLIDER/test_slider_client.py > c89e73c > ambari-server/src/test/python/stacks/2.2/configs/default.json 2fbed67 > ambari-server/src/test/python/stacks/2.2/configs/secured.json 3080043 > > Diff: https://reviews.apache.org/r/26511/diff/ > > > Testing > ------- > > Manually tested by deploying Sldier with modified definition. Ran all the > python unit tests. > > > Thanks, > > Sumit Mohanty > >
