You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Sumit Mohanty <sm...@hortonworks.com> on 2014/10/09 21:37:46 UTC

Review Request 26511: Slider service should specify what client configs can be downloaded

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26511/
-----------------------------------------------------------

Review request for Ambari 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


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 

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


Re: Review Request 26511: Slider service should specify what client configs can be downloaded

Posted by Sid Wagle <sw...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26511/#review56039
-----------------------------------------------------------

Ship it!


Ship It!

- Sid Wagle


On Oct. 9, 2014, 7:42 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:42 p.m.)
> 
> 
> Review request for Ambari 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
> 
>


Re: Review Request 26511: Slider service should specify what client configs can be downloaded

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
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
> 
>


Re: Review Request 26511: Slider service should specify what client configs can be downloaded

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
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


Re: Review Request 26511: Slider service should specify what client configs can be downloaded

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26511/
-----------------------------------------------------------

(Updated Oct. 9, 2014, 7:42 p.m.)


Review request for Ambari and Sid Wagle.


Bugs: AMBARI-7713
    https://issues.apache.org/jira/browse/AMBARI-7713


Repository: ambari


Description (updated)
-------

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


Re: Review Request 26511: Slider service should specify what client configs can be downloaded

Posted by Sumit Mohanty <sm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26511/
-----------------------------------------------------------

(Updated Oct. 9, 2014, 7:39 p.m.)


Review request for Ambari and Sid Wagle.


Changes
-------

Added unit test changes.


Bugs: AMBARI-7713
    https://issues.apache.org/jira/browse/AMBARI-7713


Repository: ambari


Description
-------

Slider service should specify what client configs can be downloaded


Diffs (updated)
-----

  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