----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37294/#review94960 -----------------------------------------------------------
Just noticed this. ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java <https://reviews.apache.org/r/37294/#comment149687> I missed this when I originally looked at the patch, sorry about that. Why was this line removed? Without this line, the components returned from the logical requests will never be added to the map. We will always have a map of hosts to empty collections. - John Speidel On Aug. 11, 2015, 5:16 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37294/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2015, 5:16 p.m.) > > > Review request for Ambari, John Speidel and Robert Nettleton. > > > Bugs: AMBARI-12688 > https://issues.apache.org/jira/browse/AMBARI-12688 > > > Repository: ambari > > > Description > ------- > > This ticket is related to > https://issues.apache.org/jira/browse/AMBARI-10750 Initial merge of advanced > api provisioning work > Detailed scenario > 1. Deploy 3-node cluster from a blueprint (NameNode is on host1. Here NN is > mapped to host1 in the cluster topology) > 2. Enable NN HA (now NameNodes are on host1 and host2) > 3. Move NN from host1 to host3(now NameNodes are on host3 and host2, but > according to cluster topology NN also mapped to host1) > 4. HDFS service check fails > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/topology/LogicalRequest.java > b7f95cf > > ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java > 8e7fb7f > ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java > 3da0fe2 > > ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java > 5d84fbc > > ambari-server/src/test/java/org/apache/ambari/server/utils/StageUtilsTest.java > 7cc70e8 > > Diff: https://reviews.apache.org/r/37294/diff/ > > > Testing > ------- > > [INFO] Ambari Server ..................................... SUCCESS > [1:14:15.296s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > > > Thanks, > > Dmytro Sen > >
