Repository: ambari
Updated Branches:
  refs/heads/branch-2.5 8b8460d7d -> e404eee44


AMBARI-15126 - Move NameNode wizard in HA environment fails to update configs 
correctly when namenode is not running on default port (rzang)


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

Branch: refs/heads/branch-2.5
Commit: e404eee4498d9031b8ec08e46cacbab353f0f30b
Parents: 8b8460d
Author: Richard Zang <[email protected]>
Authored: Wed Nov 9 18:10:18 2016 -0800
Committer: Richard Zang <[email protected]>
Committed: Wed Nov 9 18:15:06 2016 -0800

----------------------------------------------------------------------
 .../app/controllers/main/service/reassign/step4_controller.js     | 2 +-
 .../app/views/common/configs/service_configs_by_category_view.js  | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/e404eee4/ambari-web/app/controllers/main/service/reassign/step4_controller.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/app/controllers/main/service/reassign/step4_controller.js 
b/ambari-web/app/controllers/main/service/reassign/step4_controller.js
index 243fa3b..26ac68c 100644
--- a/ambari-web/app/controllers/main/service/reassign/step4_controller.js
+++ b/ambari-web/app/controllers/main/service/reassign/step4_controller.js
@@ -675,7 +675,7 @@ App.ReassignMasterWizardStep4Controller = 
App.HighAvailabilityProgressPageContro
     var ret = {};
     if (App.get('isHaEnabled')) {
       ret.namespaceId = configs['hdfs-site']['dfs.nameservices'];
-      ret.suffix = (configs['hdfs-site']['dfs.namenode.http-address.' + 
ret.namespaceId + '.nn1'] === this.get('content.reassignHosts.source') + 
':50070') ? 'nn1' : 'nn2';
+      ret.suffix = (configs['hdfs-site']['dfs.namenode.http-address.' + 
ret.namespaceId + '.nn1'].indexOf(this.get('content.reassignHosts.source')) != 
-1) ? 'nn1' : 'nn2';
     }
     return ret;
   },

http://git-wip-us.apache.org/repos/asf/ambari/blob/e404eee4/ambari-web/app/views/common/configs/service_configs_by_category_view.js
----------------------------------------------------------------------
diff --git 
a/ambari-web/app/views/common/configs/service_configs_by_category_view.js 
b/ambari-web/app/views/common/configs/service_configs_by_category_view.js
index 7710ba9..a95b441 100644
--- a/ambari-web/app/views/common/configs/service_configs_by_category_view.js
+++ b/ambari-web/app/views/common/configs/service_configs_by_category_view.js
@@ -316,7 +316,8 @@ App.ServiceConfigsByCategoryView = 
Em.View.extend(App.UserPref, App.ConfigOverri
   collapseCategory: function () {
     if (this.get('state') === 'destroyed') return;
     $('.popover').remove();
-    var filter = this.get('parentView.filter').toLowerCase();
+    var filter = this.get('parentView.filter')
+    filter = filter? filter.toLowerCase() : filter; // filter can be undefined 
in some wizard
     var filteredResult = this.get('categoryConfigs');
     var isInitialRendering = !arguments.length || arguments[1] != 
'categoryConfigs';
 

Reply via email to