You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by bh...@apache.org on 2016/03/15 22:45:00 UTC
ambari git commit: AMBARI-15396: Config recommendations for installed
services are getting cleared out (lav jain via bhuvnesh2703)
Repository: ambari
Updated Branches:
refs/heads/branch-2.2 a9bc2f4fa -> ad71694e0
AMBARI-15396: Config recommendations for installed services are getting cleared out (lav jain via bhuvnesh2703)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/ad71694e
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/ad71694e
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/ad71694e
Branch: refs/heads/branch-2.2
Commit: ad71694e01b967397ef5fde05141057b0d0f307a
Parents: a9bc2f4
Author: Bhuvnesh Chaudhary <bc...@pivotal.io>
Authored: Tue Mar 15 14:45:13 2016 -0700
Committer: Bhuvnesh Chaudhary <bc...@pivotal.io>
Committed: Tue Mar 15 14:45:13 2016 -0700
----------------------------------------------------------------------
ambari-web/app/mixins/common/configs/enhanced_configs.js | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/ad71694e/ambari-web/app/mixins/common/configs/enhanced_configs.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/common/configs/enhanced_configs.js b/ambari-web/app/mixins/common/configs/enhanced_configs.js
index 1ee4526..cd924d7 100644
--- a/ambari-web/app/mixins/common/configs/enhanced_configs.js
+++ b/ambari-web/app/mixins/common/configs/enhanced_configs.js
@@ -186,8 +186,12 @@ App.EnhancedConfigsMixin = Em.Mixin.create({
var allConfigs = stepConfigs.mapProperty('configs').filterProperty('length').reduce(function(p, c) {
return p && p.concat(c);
});
+ // Do not clear the configs that are in excluded list
+ var excludedConfigs = ['hadoop.proxyuser', 'dfs.allow.truncate'];
var cleanDependencies = this.get('_dependentConfigValues').reject(function(item) {
- if (Em.get(item, 'propertyName').contains('hadoop.proxyuser')) return false;
+ for (var i = 0; i < excludedConfigs.length; i++) {
+ if (Em.get(item, 'propertyName').contains(excludedConfigs[i])) return false;
+ }
if (installedServices.contains(Em.get(item, 'serviceName'))) {
var stackProperty = App.StackConfigProperty.find(App.config.configId(item.propertyName, item.fileName));
var parentConfigs = stackProperty && stackProperty.get('propertyDependsOn');