ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From smoha...@apache.org
Subject ambari git commit: AMBARI-19248. druid user does not have cd_access on historical nodes segment cache directory (Nishant Bangarwa via smohanty)
Date Tue, 10 Jan 2017 16:32:01 GMT
Repository: ambari
Updated Branches:
  refs/heads/trunk 0b6b67024 -> 5134c2dcc


AMBARI-19248. druid user does not have cd_access on historical nodes segment cache directory
(Nishant Bangarwa via smohanty)


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

Branch: refs/heads/trunk
Commit: 5134c2dcc8d8034dea381051518e32e7936184d3
Parents: 0b6b670
Author: Sumit Mohanty <smohanty@hortonworks.com>
Authored: Tue Jan 10 08:31:06 2017 -0800
Committer: Sumit Mohanty <smohanty@hortonworks.com>
Committed: Tue Jan 10 08:31:40 2017 -0800

----------------------------------------------------------------------
 .../DRUID/0.9.2/package/scripts/druid.py        |  2 ++
 .../test/python/stacks/2.6/DRUID/test_druid.py  | 33 +++++++++++++-------
 2 files changed, 24 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/5134c2dc/ambari-server/src/main/resources/common-services/DRUID/0.9.2/package/scripts/druid.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/common-services/DRUID/0.9.2/package/scripts/druid.py
b/ambari-server/src/main/resources/common-services/DRUID/0.9.2/package/scripts/druid.py
index 4a4f1c5..20eda92 100644
--- a/ambari-server/src/main/resources/common-services/DRUID/0.9.2/package/scripts/druid.py
+++ b/ambari-server/src/main/resources/common-services/DRUID/0.9.2/package/scripts/druid.py
@@ -194,6 +194,7 @@ def ensure_base_directories():
      params.druid_overlord_conf_dir, params.druid_router_conf_dir, params.druid_segment_infoDir,
      params.druid_tasks_dir],
     mode=0700,
+    cd_access='a',
     owner=params.druid_user,
     group=params.user_group,
     create_parents=True,
@@ -209,6 +210,7 @@ def ensure_base_directories():
       group=params.user_group,
       create_parents=True,
       recursive_ownership=True,
+      cd_access='a'
     )
 
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/5134c2dc/ambari-server/src/test/python/stacks/2.6/DRUID/test_druid.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.6/DRUID/test_druid.py b/ambari-server/src/test/python/stacks/2.6/DRUID/test_druid.py
index 104dce6..0a143ae 100644
--- a/ambari-server/src/test/python/stacks/2.6/DRUID/test_druid.py
+++ b/ambari-server/src/test/python/stacks/2.6/DRUID/test_druid.py
@@ -340,7 +340,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/_common'),
@@ -348,7 +349,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/coordinator'),
@@ -356,7 +358,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/broker'),
@@ -364,7 +367,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/middleManager'),
@@ -372,7 +376,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/historical'),
@@ -380,7 +385,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/overlord'),
@@ -388,7 +394,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', format('/usr/hdp/current/{role}/conf/router'),
@@ -396,7 +403,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', '/apps/druid/segmentCache/info_dir',
@@ -404,7 +412,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', '/apps/druid/tasks',
@@ -412,7 +421,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('Directory', '/apps/druid/segmentCache',
@@ -420,7 +430,8 @@ class TestDruid(RMFTestCase):
                               owner='druid',
                               group='hadoop',
                               create_parents=True,
-                              recursive_ownership=True
+                              recursive_ownership=True,
+                              cd_access ='a'
                               )
 
     self.assertResourceCalled('File', format('/usr/hdp/current/{role}/conf/druid-env.sh'),


Mime
View raw message