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




ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java
 (line 144)
<https://reviews.apache.org/r/51245/#comment212598>

    Can't we change the getNameNodePair() function instead to use a different 
method if the cluster is Kerberized?
    This way, we prevent future bugs from calls to getNameNodePair


- Alejandro Fernandez


On Aug. 19, 2016, 5:58 p.m., Nate Cole wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51245/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2016, 5:58 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.
> 
> 
> Bugs: AMBARI-18216
>     https://issues.apache.org/jira/browse/AMBARI-18216
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> When doing RU on a secured cluster, the MasterHostResolver class is failing 
> the lookup of JMX. Ambari's SSL configuration must be used just like metrics 
> do.
> 
> Also adding some logging, and making sure that if we have 2 confirmed NN, 
> that we are always picking one to be the active and one to be the standby.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java
>  e6154d2 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 
> 6cefa79 
>   ambari-server/src/main/java/org/apache/ambari/server/utils/HTTPUtils.java 
> cfb7128 
>   
> ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
>  55cb23b 
> 
> Diff: https://reviews.apache.org/r/51245/diff/
> 
> 
> Testing
> -------
> 
> Manual.  Automated pending.
> 
> 
> Thanks,
> 
> Nate Cole
> 
>

Reply via email to