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 2015/06/10 18:55:07 UTC

ambari git commit: AMBARI-11845. Popup during enabling HDFS ranger plugin should not happen for 2.2 stack (akovalenko)

Repository: ambari
Updated Branches:
  refs/heads/trunk 0f7130749 -> e93d40045


AMBARI-11845. Popup during enabling HDFS ranger plugin should not happen for 2.2 stack (akovalenko)


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

Branch: refs/heads/trunk
Commit: e93d40045b8bbfefb78ccac7741ef50ebf4014ae
Parents: 0f71307
Author: Aleksandr Kovalenko <ak...@hortonworks.com>
Authored: Wed Jun 10 19:48:24 2015 +0300
Committer: Aleksandr Kovalenko <ak...@hortonworks.com>
Committed: Wed Jun 10 19:54:46 2015 +0300

----------------------------------------------------------------------
 ambari-web/app/utils/configs/modification_handlers/hdfs.js | 4 ++--
 ambari-web/app/utils/configs/modification_handlers/misc.js | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/e93d4004/ambari-web/app/utils/configs/modification_handlers/hdfs.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/configs/modification_handlers/hdfs.js b/ambari-web/app/utils/configs/modification_handlers/hdfs.js
index bd18a48..ad97e22 100644
--- a/ambari-web/app/utils/configs/modification_handlers/hdfs.js
+++ b/ambari-web/app/utils/configs/modification_handlers/hdfs.js
@@ -26,13 +26,13 @@ module.exports = App.ServiceConfigModificationHandler.create({
     var newValue = changedConfig.get("value");
     var rangerPluginEnabledName = "ranger-hdfs-plugin-enabled";
     var affectedPropertyName = changedConfig.get("name");
-    if (affectedPropertyName == rangerPluginEnabledName) {
+    if (App.get('isHadoop23Stack') && affectedPropertyName == rangerPluginEnabledName) {
       var configDfsPermissionsEnabled = this.getConfig(allConfigs, 'dfs.permissions.enabled', 'hdfs-site.xml', 'HDFS');
       var configAttributesProviderClass = this.getConfig(allConfigs, 'dfs.namenode.inode.attributes.provider.class', 'hdfs-site.xml', 'HDFS');
       var isAttributesProviderClassSet = typeof configAttributesProviderClass !== 'undefined';
 
       var rangerPluginEnabled = newValue == "Yes";
-      var newDfsPermissionsEnabled = rangerPluginEnabled ? "true" : "true";
+      var newDfsPermissionsEnabled = rangerPluginEnabled ? "true" : "false";
       var newAttributesProviderClass = 'org.apache.ranger.authorization.hadoop.RangerHdfsAuthorizer';
 
       // Add HDFS-Ranger configs

http://git-wip-us.apache.org/repos/asf/ambari/blob/e93d4004/ambari-web/app/utils/configs/modification_handlers/misc.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/configs/modification_handlers/misc.js b/ambari-web/app/utils/configs/modification_handlers/misc.js
index 2af27ef..449b830 100644
--- a/ambari-web/app/utils/configs/modification_handlers/misc.js
+++ b/ambari-web/app/utils/configs/modification_handlers/misc.js
@@ -65,7 +65,7 @@ module.exports = App.ServiceConfigModificationHandler.create({
         currentUsers = currentAclValueSplits[0];
         currentGroups = currentAclValueSplits[1];
       } else {
-        currentUsers = currentAclValueSplits.length > 0 ? currentAclValueSplits.shift() : ''
+        currentUsers = currentAclValueSplits.length > 0 ? currentAclValueSplits.shift() : '';
         currentGroups = currentAclValueSplits.join(" ");
       }
       var currentUserList = currentUsers.split(',').filter(function(i) { return !Em.isEmpty(i); });