-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59838/
-----------------------------------------------------------
(Updated June 7, 2017, 12:23 p.m.)
Review request for Ambari, Alejandro Fernandez, Di Li, Jaimin Jetly, Jonathan
Hurley, Jayush Luniya, and Nate Cole.
Changes
-------
Updated patch to include documenting new functions.
Bugs: AMBARI-20122
https://issues.apache.org/jira/browse/AMBARI-20122
Repository: ambari
Description
-------
After resolution of AMBARI-19685, stack advisor validates if stack defined
dependency is not satisfied but recommendation API does not account for this.
Stack defined dependencies are service/component based and has a scope
CLUSTER|HOST.
During recommendation the services to install have already been selected. We
can't really utilize the cluster scope because either the dependent service was
selected or it was not. If it was not selected it will be caught during
validation. We can only recommend based on HOST scope.
This JIRA is also limited to only handling those which don't have conditional
dependencies.
Diffs (updated)
-----
ambari-server/src/main/resources/stacks/stack_advisor.py 4a81dc6
ambari-server/src/test/python/stacks/2.0.6/common/test_stack_advisor.py
41c57f6
Diff: https://reviews.apache.org/r/59838/diff/2/
Changes: https://reviews.apache.org/r/59838/diff/1-2/
Testing
-------
Added new test case to test the recommendation of dependencies for slaves and
masters
mvn clean test -DskipSurefireTests
New trunk without my change
Total run:1168
Total errors:13
Total failures:0
New trunk with my change
Total run:1169
Total errors:13
Total failures:0
Thanks,
Tim Thorpe
