-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46332/#review129361
-----------------------------------------------------------


Ship it!





ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
 (line 766)
<https://reviews.apache.org/r/46332/#comment192800>

    Would also accept Collections.singleton(), but thank you!


- Nate Cole


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
> 
>

Reply via email to