-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53086/
-----------------------------------------------------------
(Updated Oct. 21, 2016, 3:34 p.m.)
Review request for Ambari, Di Li, Jayush Luniya, and Juanjo Marron.
Changes
-------
Made a fix for an issue with
/home/tthorpe/dev/workspaces/slider/ambari/ambari-server/src/test/python/stacks/2.2/SLIDER/test_slider_client.py
Changed the if statement such that it avoids creating the conf directory if the
parent is a broken link in order to avoid making changes to the test code.
Bugs: AMBARI-18657
https://issues.apache.org/jira/browse/AMBARI-18657
Repository: ambari
Description
-------
The slider.py script in common-services should respect all possible ways of
installing slider. In a stack without storm there is no need to install the
storm slider client RPM. As a result the storm slider client directory will be
a broken link and attempting to create the storm slider client config directory
will result in an error.
Diffs (updated)
-----
ambari-server/src/main/resources/common-services/SLIDER/0.60.0.2.2/package/scripts/slider.py
406d076
Diff: https://reviews.apache.org/r/53086/diff/
Testing
-------
mvn clean test
and installing with and without the storm slider client RPM.
Thanks,
Tim Thorpe
