AMBARI-13700. hbase.root dir for metrics server does not get updated after NN 
HA is enabled (alexantonenko)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/014aa6dd
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/014aa6dd
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/014aa6dd

Branch: refs/heads/branch-dev-patch-upgrade
Commit: 014aa6ddb40ba8565d6bffa9c0e8afa11ec1b1ab
Parents: ddba476
Author: Alex Antonenko <[email protected]>
Authored: Wed Jan 13 17:32:22 2016 +0200
Committer: Alex Antonenko <[email protected]>
Committed: Wed Jan 13 17:59:49 2016 +0200

----------------------------------------------------------------------
 ambari-web/app/utils/configs/nn_ha_config_initializer.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/014aa6dd/ambari-web/app/utils/configs/nn_ha_config_initializer.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/configs/nn_ha_config_initializer.js 
b/ambari-web/app/utils/configs/nn_ha_config_initializer.js
index ce3e1c5..5126a3b 100644
--- a/ambari-web/app/utils/configs/nn_ha_config_initializer.js
+++ b/ambari-web/app/utils/configs/nn_ha_config_initializer.js
@@ -277,8 +277,10 @@ App.NnHaConfigInitializer = 
App.HaConfigInitializerClass.create(App.HostsBasedIn
     if (localDB.installedServices.contains('AMBARI_METRICS')) {
       var value = dependencies.serverConfigs.findProperty('type', 
'ams-hbase-site').properties['hbase.rootdir'];
       var currentNameNodeHost = 
localDB.masterComponentHosts.filterProperty('component', 
'NAMENODE').findProperty('isInstalled', true).hostName;
-      value = (value == "hdfs://" + currentNameNodeHost) ? "hdfs://" + 
dependencies.namespaceId : value;
-      configProperty.isVisible = configProperty.value != value;
+      if(value.contains("hdfs://" + currentNameNodeHost)){
+        value = value.replace(/\/\/[^\/]*/, '//' + dependencies.namespaceId);
+        configProperty.isVisible = true;
+      }
       Em.setProperties(configProperty, {
         value: value,
         recommendedValue: value

Reply via email to