----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46332/#review129342 -----------------------------------------------------------
Ship it! Ship It! - Robert Levas On April 18, 2016, 10:13 a.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46332/ > ----------------------------------------------------------- > > (Updated April 18, 2016, 10:13 a.m.) > > > Review request for Ambari, Nate Cole and Robert Levas. > > > Bugs: AMBARI-15942 > https://issues.apache.org/jira/browse/AMBARI-15942 > > > Repository: ambari > > > Description > ------- > > Caused by newly added hosts not having an ID and an internal map not being > updated correctly. > > - Deploy 2.4.0.0 cluster with Ambari 222 (secure, non-HA cluster) > - Add a new host to the cluster. The newly added host has following > components: DataNode, SmartSense HST Agent, Kerberos Client, Metrics Monitor > - Register 2.4.2.0-221 version and install the bits > - Start EU or RU > > ``` > 18 Apr 2016 05:21:37,172 ERROR [qtp-ambari-client-613] CheckHelper:120 - > Check SERVICES_UP failed > org.apache.ambari.server.HostNotFoundException: Host not found, hostname=Host > Id = 51 > at > org.apache.ambari.server.state.cluster.ClustersImpl.getHostById(ClustersImpl.java:404) > at > org.apache.ambari.server.checks.ServicesUpCheck.isConsideredDown(ServicesUpCheck.java:208) > ``` > > > Diffs > ----- > > ambari-server/src/main/java/org/apache/ambari/server/state/Clusters.java > 59dcaf8 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java > dcbf5a5 > > ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java > 186590e > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersTest.java > b563268 > > Diff: https://reviews.apache.org/r/46332/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Jonathan Hurley > >
