Browse Source

AMBARI-4179. Ambari UI should not allow to override pid dirs. (yusaku)

Yusaku Sako 11 years ago
parent
commit
873e9b34b0

+ 4 - 0
ambari-web/app/data/HDP2/global_properties.js

@@ -280,6 +280,7 @@ module.exports =
       "description": "",
       "defaultValue": "/var/log/hadoop-mapreduce",
       "displayType": "directory",
+      "isOverridable": false,
       "isVisible": true,
       "isReconfigurable": false,
       "serviceName": "MAPREDUCE2",
@@ -292,6 +293,7 @@ module.exports =
       "description": "",
       "defaultValue": "/var/run/hadoop-mapreduce",
       "displayType": "directory",
+      "isOverridable": false,
       "isVisible": true,
       "isReconfigurable": false,
       "serviceName": "MAPREDUCE2",
@@ -376,6 +378,7 @@ module.exports =
       "description": "",
       "defaultValue": "/var/log/hadoop-yarn",
       "displayType": "directory",
+      "isOverridable": false,
       "isReconfigurable": false,
       "isVisible": true,
       "serviceName": "YARN",
@@ -388,6 +391,7 @@ module.exports =
       "description": "",
       "defaultValue": "/var/run/hadoop-yarn",
       "displayType": "directory",
+      "isOverridable": false,
       "isVisible": true,
       "isReconfigurable": false,
       "serviceName": "YARN",

+ 1 - 1
ambari-web/app/models/service_config.js

@@ -180,7 +180,7 @@ App.ServiceConfigProperty = Ember.Object.extend({
    * and fields which represent master hosts.
    */
   isPropertyOverridable: function () {
-    var overrideable = this.get('name').contains('pid_dir') ? false : this.get('isOverridable');
+    var overrideable = this.get('isOverridable');
     var editable = this.get('isEditable');
     var overrides = this.get('overrides');
     var dt = this.get('displayType');