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

Ship it!


Ship It!

- Tom Beerbower


On Feb. 12, 2015, 7:06 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30879/
> -----------------------------------------------------------
> 
> (Updated Feb. 12, 2015, 7:06 p.m.)
> 
> 
> Review request for Ambari and John Speidel.
> 
> 
> Bugs: AMBARI-9566
>     https://issues.apache.org/jira/browse/AMBARI-9566
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> PROBLEM:
> 
> HDP 2.2.0, Ambari 1.7.0.  
> When enable namnode HA with https by dfs.http.policy=HTTPS_ONLY, and set up
> required ssl-server.xml and ssl-client.xml, hdfs function fine.
> 
> But Ambari UI shows no metrics for namenode, even shows 'upgrade not
> finalized' when it is a fresh HDP 2.2.0. install.
> 
> A few notes:  
> 1\. there is no option in 'ambari-server setup-security' or documentation on
> how to set up trust store for Ambari to trust hdfs/yarn
> 
> 2\. it looks like Ambari still uses hadoop.ssl.enabled as criteria whether
> https is enabled. This is deprecated since hdp 2.1. The property should be
> dfs.http.policy, yarn.http.policy.
> 
> screenshot attached that shows the wrong and missing metrics, and screenshot
> of the jmx from namenode
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
>  4c47a1c 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXHostProvider.java
>  65f7be7 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/jmx/JMXPropertyProvider.java
>  bbacb0d 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration/yarn-site.xml
>  6572935 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/JMXHostProviderTest.java
>  d8e478a 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/metrics/JMXPropertyProviderTest.java
>  69fb5f5 
>   ambari-web/app/models/service_config.js 999d1b5 
> 
> Diff: https://reviews.apache.org/r/30879/diff/
> 
> 
> Testing
> -------
> 
> [INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 
> approved: 41 licence.
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Ambari Main ....................................... SUCCESS [8.873s]
> [INFO] Apache Ambari Project POM ......................... SUCCESS [0.365s]
> [INFO] Ambari Web ........................................ SUCCESS [1:01.055s]
> [INFO] Ambari Views ...................................... SUCCESS [5.658s]
> [INFO] Ambari Admin View ................................. SUCCESS [25.783s]
> [INFO] Ambari Server ..................................... SUCCESS 
> [33:27.424s]
> [INFO] Ambari Agent ...................................... SUCCESS [11.128s]
> [INFO] Ambari Client ..................................... SUCCESS [0.063s]
> [INFO] Ambari Python Client .............................. SUCCESS [0.372s]
> [INFO] Ambari Groovy Client .............................. SUCCESS [11.368s]
> [INFO] Ambari Shell ...................................... SUCCESS [0.066s]
> [INFO] Ambari Python Shell ............................... SUCCESS [0.093s]
> [INFO] Ambari Groovy Shell ............................... SUCCESS [8.573s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 35:41.785s
> [INFO] Finished at: Thu Feb 12 21:01:52 EET 2015
> [INFO] Final Memory: 61M/544M
> [INFO] -----------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>

Reply via email to