AMBARI-4162. Ambari trying to setup namenode dirs on nfs mounted home dirs. (onechiporenko)

ambari-web/app/models/service_config.js

@@ -229,7 +229,6 @@ App.ServiceConfigProperty = Ember.Object.extend({
     types.forEach(function(type) {
       if (type === displayType) {
         result = true;
-        return;
       }
     });
     return result;
@@ -664,16 +663,17 @@ App.ServiceConfigProperty = Ember.Object.extend({
         case 'checkbox':
           break;
         case 'directories':
-          if (!validator.isValidDir(value)) {
-            this.set('errorMessage', 'Must be a slash at the start');
-            isError = true;
-          }
-          break;
         case 'directory':
           if (!validator.isValidDir(value)) {
             this.set('errorMessage', 'Must be a slash at the start');
             isError = true;
           }
+          else {
+            if (!validator.isAllowedDir(value)) {
+              this.set('errorMessage', 'Can\'t start with "home(s)"');
+              isError = true;
+            }
+          }
           break;
         case 'custom':
           break;
@@ -691,7 +691,7 @@ App.ServiceConfigProperty = Ember.Object.extend({
           break;
         case 'host':
           var hiveOozieHostNames = ['hive_hostname','hive_existing_mysql_host','hive_existing_oracle_host','hive_ambari_host',
-          'oozie_hostname','oozie_existing_mysql_host','oozie_existing_oracle_host','oozie_ambari_host']
+          'oozie_hostname','oozie_existing_mysql_host','oozie_existing_oracle_host','oozie_ambari_host'];
           if(hiveOozieHostNames.contains(this.get('name'))) {
             if (validator.hasSpaces(value)) {
               this.set('errorMessage', Em.I18n.t('host.spacesValidation'));

ambari-web/app/utils/validator.js

@@ -56,6 +56,21 @@ module.exports = {
     return true;
   },
 
+  /**
+   * validate directory doesn't start "home" or "homes"
+   * @param value
+   * @returns {boolean}
+   */
+  isAllowedDir: function(value) {
+    var dirs = value.replace(/,/g,' ').trim().split(new RegExp("\\s+", "g"));
+    for(var i = 0; i < dirs.length; i++){
+      if(dirs[i].startsWith('/home') || dirs[i].startsWith('/homes')) {
+        return false;
+      }
+    }
+    return true;
+  },
+
   /**
    * validate ip address with port
    * @param value

ambari-web/test/utils/validator_test.js

@@ -256,6 +256,24 @@ describe('validator', function () {
       })
     });
   })
+  describe('#isAllowedDir(value)', function() {
+    var tests = [
+      {m:'"/home" - not allowed',i:'/home',e:false},
+      {m:'"/homes" - not allowed',i:'/homes',e:false},
+      {m:'"/home/" - not allowed',i:'/home/',e:false},
+      {m:'"/homes/" - not allowed',i:'/homes/',e:false},
+      {m:'"/dir" - allowed',i:'/dir',e:true},
+      {m:'"/dir/home" - allowed',i:'/dir/home',e:true},
+      {m:'"/dir/homes" - allowed',i:'/dir/homes',e:true},
+      {m:'"/dir/home/" - allowed',i:'/dir/home/',e:true},
+      {m:'"/dir/homes/" - allowed',i:'/dir/homes/',e:true}
+    ];
+    tests.forEach(function(test) {
+      it(test.m + ' ', function () {
+        expect(validator.isAllowedDir(test.i)).to.equal(test.e);
+      })
+    });
+  })
   describe('#isValidConfigKey(value)', function() {
     var tests = [
       {m:'"123" - valid',i:'123',e:true},