----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47701/ -----------------------------------------------------------
Review request for Ambari, Dmytro Sen and Sid Wagle. Summary (updated) ----------------- After enabling HTTPS for YARN, NodeManager JVM Metrics on YARN Heatmaps show NA. Bugs: AMBARI-16642 https://issues.apache.org/jira/browse/AMBARI-16642 Repository: ambari Description (updated) ------- After enabling HTTPS for YARN, "NodeManager GC Time" and "NodeManager JVM Heap Memory Used" widgets on YARN Heatmaps show NA. The root cause is that yarn.nodemanager.webapp.address rather than yarn.nodemanager.webapp.https.address is used to get JMX metrics. Steps to reproduce: 1. install a cluster with default from Ambari Web UI. 2. configuring SSL for HDFS, YARN, and MapReduce. 3. enable HTTPS for YARN 1). Set the following property in Advanced yarn-site from Ambari Web UI: yarn.http.policy=HTTPS_ONLY 2). Add the following property in Custom yarn-site from Ambari Web UI: yarn.nodemanager.webapp.https.address=0.0.0.0:8044 4. "NodeManager GC Time" and "NodeManager JVM Heap Memory Used" widgets on YARN Heatmaps will show NA. Diffs ----- Diff: https://reviews.apache.org/r/47701/diff/ Testing (updated) ------- 1. The fix has been tested with HDP UI "NodeManager GC Time" and "NodeManager JVM Heap Memory Used" metrics show up after apply the fix 2. JMXHostProviderTest.java has been modified to verify JMX protocol and port for NodeManager. Test set: org.apache.ambari.server.controller.internal.JMXHostProviderTest ------------------------------------------------------------------------------- Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 120.274 sec - in org.apache.ambari.server.controller.internal.JMXHostProviderTest File Attachments (updated) ---------------- AMBARI-16642.patch https://reviews.apache.org/media/uploaded/files/2016/05/22/f40fa1d3-53e0-4e52-988a-b1b128d01cfc__AMBARI-16642.patch Thanks, Qin Liu