ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From onechipore...@apache.org
Subject git commit: AMBARI-6402. Many services failed to start when using custom user names and groups. (onechiporenko)
Date Mon, 07 Jul 2014 17:52:25 GMT
Repository: ambari
Updated Branches:
  refs/heads/trunk 510ff0a6a -> 67c40e3f5


AMBARI-6402. Many services failed to start when using custom user names and groups. (onechiporenko)


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

Branch: refs/heads/trunk
Commit: 67c40e3f5e53e72b80f43e7df1a11c070fdaadea
Parents: 510ff0a
Author: Oleg Nechiporenko <onechiporenko@apache.org>
Authored: Mon Jul 7 20:12:45 2014 +0300
Committer: Oleg Nechiporenko <onechiporenko@apache.org>
Committed: Mon Jul 7 20:52:15 2014 +0300

----------------------------------------------------------------------
 ambari-web/app/utils/config.js | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/67c40e3f/ambari-web/app/utils/config.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/config.js b/ambari-web/app/utils/config.js
index 8378c57..cd6e901 100644
--- a/ambari-web/app/utils/config.js
+++ b/ambari-web/app/utils/config.js
@@ -462,16 +462,20 @@ App.config = Em.Object.create({
           configData.isRequired = (configData.isRequired !== undefined) ? configData.isRequired
: true;
           configData.isRequiredByAgent = (configData.isRequiredByAgent !== undefined) ? configData.isRequiredByAgent
: true;
           configData.showLabel = stored.showLabel !== false;
-        } else if (!preDefined && stored) {
-          this.addUserProperty(stored, isAdvanced, advancedConfigs);
-        } else if (preDefined && !stored) {
-          configData = preDefined;
-          configData.isRequiredByAgent = (configData.isRequiredByAgent !== undefined) ? configData.isRequiredByAgent
: true;
-          if (isAdvanced) {
-            var advanced = advancedConfigs.findProperty('name', configData.name);
-            this.setPropertyFromStack(configData,advanced);
-          }
         }
+        else
+          if (!preDefined && stored) {
+            configData = this.addUserProperty(stored, isAdvanced, advancedConfigs);
+          }
+          else
+            if (preDefined && !stored) {
+              configData = preDefined;
+              configData.isRequiredByAgent = (configData.isRequiredByAgent !== undefined)
? configData.isRequiredByAgent : true;
+              if (isAdvanced) {
+                var advanced = advancedConfigs.findProperty('name', configData.name);
+                this.setPropertyFromStack(configData,advanced);
+              }
+            }
         
         if (configData.displayType === 'checkbox') {
           configData.value = configData.value === 'true'; // convert {String} value to {Boolean}


Mime
View raw message