-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35389/
-----------------------------------------------------------
Review request for Ambari and Vitalyi Brodetskyi.
Bugs: AMBARI-11885
https://issues.apache.org/jira/browse/AMBARI-11885
Repository: ambari
Description
-------
After upgrading to HDP 2.2.4.2 we removed the HDP 2.2.0 directories to make
sure we didn't have any hidden dependencies. When restarting services,
supervisors failed to start with this error:
Fail: Execution of 'ambari-sudo.sh ln -s /usr/lib/storm/lib/ambari-metrics-
storm-sink*.jar /usr/hdp/current/storm-client/lib/ambari-metrics-storm-
sink.jar' returned 1. ln: creating symbolic link `/usr/hdp/current/storm-
client/lib/ambari-metrics-storm-sink.jar': No such file or directory
2015-05-22 13:33:27,950 - Command: /usr/bin/hdp-select status storm-supervisor
> /tmp/tmpXh9erp
We removed the supervisors and redeployed them, thinking it might be solved by
a fresh deployment. When starting the newly deployed supervisors, we got this
error:
Fail: Execution of 'ambari-sudo.sh ln -s /usr/lib/storm/lib/ambari-metrics-
storm-sink*.jar /usr/hdp/current/storm-client/lib/ambari-metrics-storm-
sink.jar' returned 1. ln: creating symbolic link `/usr/hdp/current/storm-
client/lib/ambari-metrics-storm-sink.jar': No such file or directory
2015-05-22 13:36:59,979 - Command: /usr/bin/hdp-select status storm-supervisor
> /tmp/tmp4iP41d
When a supervisor is deployed (in the absence of the nimbus, DRPC server or
UI) it does create a /usr/hdp/current/storm-supervisor link that correctly
links to /usr/hdp/<latestVersion>/storm, but it does not create a link for
storm-client. Ambari Metrics tries to create the link for its jar using the
storm-client link.
Ambari Metrics needs to change the path for its link to use the appropriate
component (in this case storm-supervisor).
Diffs
-----
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/params_linux.py
68f2d0e
Diff: https://reviews.apache.org/r/35389/diff/
Testing
-------
mvn clean test
Thanks,
Andrew Onischuk
