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 <[email protected]> Authored: Tue Jan 10 08:31:06 2017 -0800 Committer: Sumit Mohanty <[email protected]> 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'),
