ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aonis...@apache.org
Subject ambari git commit: AMBARI-13808. Ambari does not change permissions and owner/group for log and PID directories during install (aonishuk)
Date Tue, 10 Nov 2015 12:30:46 GMT
Repository: ambari
Updated Branches:
  refs/heads/branch-2.1 165164870 -> 570a7d538


AMBARI-13808. Ambari does not change permissions and owner/group for log and PID directories
during install (aonishuk)


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

Branch: refs/heads/branch-2.1
Commit: 570a7d538c4eecd90f7e9d2c4024bcc641c09575
Parents: 1651648
Author: Andrew Onishuk <aonishuk@hortonworks.com>
Authored: Tue Nov 10 14:30:40 2015 +0200
Committer: Andrew Onishuk <aonishuk@hortonworks.com>
Committed: Tue Nov 10 14:30:40 2015 +0200

----------------------------------------------------------------------
 .../package/scripts/accumulo_configuration.py   |  8 ++++--
 .../AMBARI_METRICS/0.1.0/package/scripts/ams.py |  5 +++-
 .../FLUME/1.4.0.2.0/package/scripts/flume.py    |  6 ++++-
 .../HBASE/0.96.0.2.0/package/scripts/hbase.py   |  8 ++++--
 .../STORM/0.9.1.2.1/package/scripts/storm.py    |  1 +
 .../3.4.5.2.0/package/scripts/zookeeper.py      |  9 ++++---
 .../AMBARI_METRICS/test_metrics_collector.py    |  3 ++-
 .../python/stacks/2.0.6/FLUME/test_flume.py     | 15 ++++++++---
 .../stacks/2.0.6/HBASE/test_hbase_master.py     | 18 +++++++++++--
 .../2.0.6/HBASE/test_hbase_regionserver.py      | 28 +++++++++++++++++---
 .../2.0.6/HBASE/test_phoenix_queryserver.py     |  8 ++++++
 .../2.0.6/ZOOKEEPER/test_zookeeper_client.py    | 10 +++++--
 .../2.0.6/ZOOKEEPER/test_zookeeper_server.py    | 10 +++++--
 .../python/stacks/2.1/STORM/test_storm_base.py  | 12 ++++++---
 14 files changed, 113 insertions(+), 28 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py
b/ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py
index d30a460..acc340f 100644
--- a/ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py
+++ b/ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py
@@ -85,14 +85,18 @@ def setup_conf_dir(name=None): # 'master' or 'tserver' or 'monitor' or
'gc' or '
     Directory( params.pid_dir,
                owner = params.accumulo_user,
                group = params.user_group,
-               recursive = True
+               recursive = True,
+               cd_access = "a",
+               mode = 0755,
     )
 
     # create log dir
     Directory (params.log_dir,
                owner = params.accumulo_user,
                group = params.user_group,
-               recursive = True
+               recursive = True,
+               cd_access = "a",
+               mode = 0755,
     )
 
     # create env file

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py
b/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py
index caac937..50e363c 100644
--- a/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py
+++ b/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py
@@ -248,7 +248,8 @@ def ams(name=None):
               owner=params.ams_user,
               group=params.user_group,
               cd_access="a",
-              recursive=True
+              recursive=True,
+              mode=0755,
     )
 
     # Hack to allow native HBase libs to be included for embedded hbase
@@ -333,12 +334,14 @@ def ams(name=None):
     Directory(params.ams_monitor_log_dir,
               owner=params.ams_user,
               group=params.user_group,
+              mode=0755,
               recursive=True
     )
 
     Directory(params.ams_monitor_pid_dir,
               owner=params.ams_user,
               group=params.user_group,
+              mode=0755,
               recursive=True
     )
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py
b/ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py
index b617313..293df58 100644
--- a/ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py
+++ b/ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py
@@ -99,7 +99,11 @@ def flume(action = None):
               recursive=True,
               owner=params.flume_user,
               )
-    Directory(params.flume_log_dir, owner=params.flume_user)
+    Directory(params.flume_log_dir, 
+              owner=params.flume_user,
+              cd_access="a",
+              mode=0755,
+    )
 
     flume_agents = {}
     if params.flume_conf_content is not None:

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
index 4d2aaf8..2b2cf1e 100644
--- a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
+++ b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
@@ -154,12 +154,16 @@ def hbase(name=None):
   if name != "client":
     Directory( params.pid_dir,
       owner = params.hbase_user,
-      recursive = True
+      recursive = True,
+      cd_access = "a",
+      mode = 0755,
     )
   
     Directory (params.log_dir,
       owner = params.hbase_user,
-      recursive = True
+      recursive = True,
+      cd_access = "a",
+      mode = 0755,
     )
 
   if (params.log4j_props != None):

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py
b/ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py
index 699a868..51998fa 100644
--- a/ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py
+++ b/ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py
@@ -64,6 +64,7 @@ def storm(name=None):
             group=params.user_group,
             recursive=True,
             cd_access="a",
+            mode=0755,
   )
 
   Directory(params.conf_dir,

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py
b/ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py
index c78acd9..e9f44cc 100644
--- a/ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py
+++ b/ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py
@@ -59,20 +59,23 @@ def zookeeper(type = None, upgrade_type=None):
   Directory(params.zk_pid_dir,
             owner=params.zk_user,
             recursive=True,
-            group=params.user_group
+            group=params.user_group,
+            mode=0755,
   )
 
   Directory(params.zk_log_dir,
             owner=params.zk_user,
             recursive=True,
-            group=params.user_group
+            group=params.user_group,
+            mode=0755,
   )
 
   Directory(params.zk_data_dir,
             owner=params.zk_user,
             recursive=True,
             cd_access="a",
-            group=params.user_group
+            group=params.user_group,
+            mode=0755,
   )
 
   if type == 'server':

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py
b/ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py
index 7add394..4dd2058 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py
@@ -108,7 +108,8 @@ class TestOozieClient(RMFTestCase):
                               owner = 'ams',
                               cd_access = 'a',
                               group = 'hadoop',
-                              recursive = True
+                              recursive = True,
+                              mode=0755,
     )
     self.assertResourceCalled('File', '/usr/lib/ams-hbase/bin/hadoop',
                               owner = 'ams',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py b/ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py
index ee9377f..65ba8fa 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py
@@ -222,7 +222,10 @@ class TestFlumeHandler(RMFTestCase):
 
     self.assertResourceCalled('Directory',
                               '/var/log/flume',
-                              owner = 'flume')
+                              owner = 'flume',
+                              cd_access = 'a', 
+                              mode=0755
+    )
 
     self.assertResourceCalled('Directory',
                               '/etc/flume/conf/a1',
@@ -265,7 +268,7 @@ class TestFlumeHandler(RMFTestCase):
                               owner='flume',
                               recursive=True)
 
-    self.assertResourceCalled('Directory', '/var/log/flume', owner = 'flume')
+    self.assertResourceCalled('Directory', '/var/log/flume', owner = 'flume', cd_access =
'a', mode=0755)
 
     top = build_flume(self.getConfig()['configurations']['flume-conf']['content'])
 
@@ -430,7 +433,9 @@ class TestFlumeHandler(RMFTestCase):
 
     self.assertResourceCalled('Directory',
                               '/var/log/flume',
-                              owner = 'flume')
+                              owner = 'flume',
+                              cd_access = 'a',
+                              mode=0755)
 
     self.assertResourceCalled('Directory',
                               '/etc/flume/conf/a1',
@@ -479,7 +484,9 @@ class TestFlumeHandler(RMFTestCase):
 
     self.assertResourceCalled('Directory',
                               '/var/log/flume',
-                              owner = 'flume')
+                              owner = 'flume', 
+                              cd_access = 'a', 
+                              mode=0755)
 
     self.assertResourceCalled('Directory',
                               '/usr/hdp/current/flume-server/conf/a1',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
index 911fd1f..3206e1c 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
@@ -307,10 +307,14 @@ class TestHBaseMaster(RMFTestCase):
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('File',
                               '/etc/hbase/conf/log4j.properties',
@@ -441,10 +445,14 @@ class TestHBaseMaster(RMFTestCase):
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('File',
                               '/etc/hbase/conf/log4j.properties',
@@ -580,11 +588,17 @@ class TestHBaseMaster(RMFTestCase):
 
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
-      recursive = True)
+      recursive = True,
+      mode = 0755,
+      cd_access = 'a',
+    )
 
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
-      recursive = True)
+      recursive = True,
+      mode = 0755,
+      cd_access = 'a',
+    )
 
     self.assertResourceCalled('File',
                               '/usr/hdp/current/hbase-master/conf/log4j.properties',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py
index f8d5545..84e5753 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py
@@ -207,10 +207,14 @@ class TestHbaseRegionServer(RMFTestCase):
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('File',
                               '/etc/hbase/conf/log4j.properties',
@@ -305,10 +309,14 @@ class TestHbaseRegionServer(RMFTestCase):
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('File',
                               '/etc/hbase/conf/log4j.properties',
@@ -412,11 +420,17 @@ class TestHbaseRegionServer(RMFTestCase):
 
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
-      recursive = True)
+      recursive = True,
+      mode = 0755,
+      cd_access = 'a',
+    )
 
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
-      recursive = True)
+      recursive = True,
+      mode = 0755,
+      cd_access = 'a',
+    )
 
     self.assertResourceCalled('File',
                               '/usr/hdp/current/hbase-regionserver/conf/log4j.properties',
@@ -524,11 +538,17 @@ class TestHbaseRegionServer(RMFTestCase):
 
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
-      recursive = True)
+      recursive = True,
+      mode = 0755,
+      cd_access = 'a',
+    )
 
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
-      recursive = True)
+      recursive = True,
+      mode = 0755,
+      cd_access = 'a',
+    )
 
     self.assertResourceCalled('File',
                               '/usr/hdp/current/hbase-regionserver/conf/log4j.properties',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py
b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py
index 256b478..ddb111e 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py
@@ -300,10 +300,14 @@ class TestPhoenixQueryServer(RMFTestCase):
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('File',
       '/usr/hdp/current/hbase-regionserver/conf/log4j.properties',
@@ -412,10 +416,14 @@ class TestPhoenixQueryServer(RMFTestCase):
     self.assertResourceCalled('Directory', '/var/run/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('Directory', '/var/log/hbase',
       owner = 'hbase',
       recursive = True,
+      mode = 0755,
+      cd_access = 'a',
     )
     self.assertResourceCalled('File',
       '/usr/hdp/current/hbase-regionserver/conf/log4j.properties',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py
b/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py
index 120f895..a77b25d 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py
@@ -62,17 +62,20 @@ class TestZookeeperClient(RMFTestCase):
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/var/log/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/hadoop/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode = 0755,
     )
     self.assertResourceCalled('File',
                               '/etc/zookeeper/conf/log4j.properties',
@@ -122,17 +125,20 @@ class TestZookeeperClient(RMFTestCase):
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/var/log/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/hadoop/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode = 0755
     )
     self.assertResourceCalled('File',
                               '/etc/zookeeper/conf/log4j.properties',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py
b/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py
index fccc19e..5cc5760 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py
@@ -142,17 +142,20 @@ class TestZookeeperServer(RMFTestCase):
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/var/log/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/hadoop/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode = 0755,
     )
     self.assertResourceCalled('File', '/hadoop/zookeeper/myid',
       content = '1',
@@ -197,17 +200,20 @@ class TestZookeeperServer(RMFTestCase):
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/var/log/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
+      mode = 0755,
     )
     self.assertResourceCalled('Directory', '/hadoop/zookeeper',
       owner = 'zookeeper',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode = 0755,
     )
     self.assertResourceCalled('File', '/hadoop/zookeeper/myid',
       content = '1',

http://git-wip-us.apache.org/repos/asf/ambari/blob/570a7d53/ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py b/ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py
index daf4bfe..301bb58 100644
--- a/ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py
+++ b/ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py
@@ -40,13 +40,15 @@ class TestStormBase(RMFTestCase):
       owner = 'storm',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode=0755,
     )
     self.assertResourceCalled('Directory', '/hadoop/storm',
       owner = 'storm',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode=0755,
     )
     self.assertResourceCalled('Directory', confDir,
       group = 'hadoop',
@@ -81,13 +83,15 @@ class TestStormBase(RMFTestCase):
       owner = 'storm',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode=0755,
     )
     self.assertResourceCalled('Directory', '/hadoop/storm',
       owner = 'storm',
       group = 'hadoop',
       recursive = True,
-      cd_access='a'
+      cd_access='a',
+      mode=0755,
     )
     self.assertResourceCalled('Directory', confDir,
       group = 'hadoop',


Mime
View raw message