You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ak...@apache.org on 2016/02/25 10:27:33 UTC

ambari git commit: AMBARI-15164. Hide "Add Override" buttons in NameNode HA Wizard on step3 (akovalenko)

Repository: ambari
Updated Branches:
  refs/heads/trunk 16a0d794b -> 0abc53fa5


AMBARI-15164. Hide "Add Override" buttons in NameNode HA Wizard on step3 (akovalenko)


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

Branch: refs/heads/trunk
Commit: 0abc53fa55767e72229d783aa862813e51ffd067
Parents: 16a0d79
Author: Aleksandr Kovalenko <ak...@hortonworks.com>
Authored: Wed Feb 24 18:10:32 2016 +0200
Committer: Aleksandr Kovalenko <ak...@hortonworks.com>
Committed: Thu Feb 25 11:27:22 2016 +0200

----------------------------------------------------------------------
 .../main/admin/highAvailability/nameNode/step3_controller.js | 1 +
 .../admin/highAvailability/nameNode/step3_controller_test.js | 8 ++++++++
 2 files changed, 9 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/0abc53fa/ambari-web/app/controllers/main/admin/highAvailability/nameNode/step3_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/admin/highAvailability/nameNode/step3_controller.js b/ambari-web/app/controllers/main/admin/highAvailability/nameNode/step3_controller.js
index 4e4df60..2df1ef4 100644
--- a/ambari-web/app/controllers/main/admin/highAvailability/nameNode/step3_controller.js
+++ b/ambari-web/app/controllers/main/admin/highAvailability/nameNode/step3_controller.js
@@ -172,6 +172,7 @@ App.HighAvailabilityWizardStep3Controller = Em.Controller.extend({
 
     configs.forEach(function (config) {
       App.NnHaConfigInitializer.initialValue(config, localDB, dependencies);
+      config.isOverridable = false;
     });
 
     return configs;

http://git-wip-us.apache.org/repos/asf/ambari/blob/0abc53fa/ambari-web/test/controllers/main/admin/highAvailability/nameNode/step3_controller_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/controllers/main/admin/highAvailability/nameNode/step3_controller_test.js b/ambari-web/test/controllers/main/admin/highAvailability/nameNode/step3_controller_test.js
index 8067320..d4c2e41 100644
--- a/ambari-web/test/controllers/main/admin/highAvailability/nameNode/step3_controller_test.js
+++ b/ambari-web/test/controllers/main/admin/highAvailability/nameNode/step3_controller_test.js
@@ -391,6 +391,14 @@ describe('App.HighAvailabilityWizardStep3Controller', function() {
       });
     });
 
+    it('should set isOverridable=false for each config', function () {
+      var configs = [
+        {name: 'prop1'}, {name: 'prop2'}
+      ];
+      configs = controller.tweakServiceConfigs(configs);
+      expect(configs.everyProperty('isOverridable', false)).to.be.true;
+    });
+
   });
 
 });