-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51815/
-----------------------------------------------------------
(Updated Nov. 1, 2016, 6:57 p.m.)
Review request for Ambari, Shantanu Mundkur, Di Li, Jayush Luniya, Juanjo
Marron, Laszlo Puskas, Nate Cole, and Robert Nettleton.
Changes
-------
Hello Nate, Could you please review the updated patch. Thank you
Bugs: AMBARI-18355
https://issues.apache.org/jira/browse/AMBARI-18355
Repository: ambari
Description
-------
Currently stack definitions do not list conditional dependencies, adding those
to the stack definitions would make it easy to validate errors in case of
blueprint deployment. Please refer to document attached to Jira
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/state/DependencyConditionInfo.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/state/DependencyInfo.java
e3db662
ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java
a5f33ff
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml
9c1387d
ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintImplTest.java
ff9af17
ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintValidatorImplTest.java
b1de8ef
Diff: https://reviews.apache.org/r/51815/diff/
Testing
-------
Written Junit test cases. Perfomred manual testing to check the Namenode HA
component dependency. Was able to proceed with the installation for valid
blueprint. and got validation error message while registering blueprint when
the Blueprint did not satisfy the conditional dependencies.
Thanks,
Amruta Borkar
