You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by sm...@apache.org on 2017/01/10 16:31:18 UTC
ambari git commit: AMBARI-19248. druid user does not have cd_access
on historical nodes segment cache directory (Nishant Bangarwa via smohanty)
Repository: ambari
Updated Branches:
refs/heads/branch-2.5 6127c77c9 -> b1bfb25fe
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/b1bfb25f
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/b1bfb25f
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/b1bfb25f
Branch: refs/heads/branch-2.5
Commit: b1bfb25fe4ac5e61041bb1ed046a63b97f77c31f
Parents: 6127c77
Author: Sumit Mohanty <sm...@hortonworks.com>
Authored: Tue Jan 10 08:31:06 2017 -0800
Committer: Sumit Mohanty <sm...@hortonworks.com>
Committed: Tue Jan 10 08:31:06 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/b1bfb25f/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/b1bfb25f/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'),