Repository: ambari Updated Branches: refs/heads/trunk 14587d6eb -> 14c13c1c6
AMBARI-13855. Ambari does not change permissions and owner/group for log and PID directories during install on cluster with UMASK 027 + non-root agent (aonishuk) Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/14c13c1c Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/14c13c1c Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/14c13c1c Branch: refs/heads/trunk Commit: 14c13c1c6419c94a3e51da9cc592079e24f04067 Parents: 14587d6 Author: Andrew Onishuk <[email protected]> Authored: Thu Nov 12 19:01:09 2015 +0200 Committer: Andrew Onishuk <[email protected]> Committed: Thu Nov 12 19:01:09 2015 +0200 ---------------------------------------------------------------------- .../AMBARI_METRICS/0.1.0/package/scripts/ams.py | 3 +- .../0.1.0/package/scripts/hbase.py | 8 +++- .../FALCON/0.5.0.2.1/package/scripts/falcon.py | 10 ++++- .../KNOX/0.5.0.2.2/package/scripts/knox.py | 4 +- .../AMBARI_METRICS/test_metrics_collector.py | 13 +++++-- .../stacks/2.1/FALCON/test_falcon_client.py | 8 +++- .../stacks/2.1/FALCON/test_falcon_server.py | 12 +++++- .../python/stacks/2.2/KNOX/test_knox_gateway.py | 40 +++++++++++++++----- 8 files changed, 74 insertions(+), 24 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/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 50e363c..0bdb45f 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 @@ -241,7 +241,8 @@ def ams(name=None): owner=params.ams_user, group=params.user_group, cd_access="a", - recursive=True + recursive=True, + mode=0755, ) Directory(params.ams_collector_pid_dir, http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py b/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py index 9e6f2fa..a1c4581 100644 --- a/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py +++ b/ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py @@ -176,12 +176,16 @@ def hbase(name=None # 'master' or 'regionserver' or 'client' if name != "client": Directory( params.hbase_pid_dir, owner = params.hbase_user, - recursive = True + recursive = True, + cd_access = "a", + mode = 0755, ) Directory (params.hbase_log_dir, owner = params.hbase_user, - recursive = True + recursive = True, + cd_access = "a", + mode = 0755, ) if name == "master": http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py b/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py index aacb988..6c8d5ff 100644 --- a/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py +++ b/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py @@ -41,11 +41,17 @@ def falcon(type, action = None, upgrade_type=None): if action == 'config': Directory(params.falcon_pid_dir, owner = params.falcon_user, - recursive = True) + recursive = True, + mode = 0755, + cd_access = "a", + ) Directory(params.falcon_log_dir, owner = params.falcon_user, - recursive = True) + recursive = True, + mode = 0755, + cd_access = "a", + ) Directory(params.falcon_webapp_dir, owner = params.falcon_user, http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox.py b/ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox.py index bb0bbfe..2a83952 100644 --- a/ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox.py +++ b/ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox.py @@ -93,7 +93,9 @@ def knox(): Directory(directory, owner = params.knox_user, group = params.knox_group, - recursive = True + recursive = True, + cd_access = "a", + mode = 0755, ) XmlConfig("gateway-site.xml", http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/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 b04e1d5..211c417 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 @@ -22,7 +22,7 @@ from stacks.utils.RMFTestCase import * @patch("os.path.exists", new = MagicMock(return_value=True)) @patch("platform.linux_distribution", new = MagicMock(return_value="Linux")) -class TestOozieClient(RMFTestCase): +class TestMetricsCollector(RMFTestCase): COMMON_SERVICES_PACKAGE_DIR = "AMBARI_METRICS/0.1.0/package" STACK_VERSION = "2.0.6" @@ -124,7 +124,8 @@ class TestOozieClient(RMFTestCase): owner = 'ams', group = 'hadoop', cd_access = 'a', - recursive = True + recursive = True, + mode = 0755, ) self.assertResourceCalled('Directory', '/var/run/ambari-metrics-collector', owner = 'ams', @@ -229,11 +230,15 @@ class TestOozieClient(RMFTestCase): ) self.assertResourceCalled('Directory', '/var/run/ambari-metrics-collector/', owner = 'ams', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/log/ambari-metrics-collector', owner = 'ams', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) if name == 'master': http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py b/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py index ba9f920..685eb50 100644 --- a/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py +++ b/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_client.py @@ -37,11 +37,15 @@ class TestFalconClient(RMFTestCase): ) self.assertResourceCalled('Directory', '/var/run/falcon', owner = 'falcon', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/log/falcon', owner = 'falcon', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/lib/falcon/webapp', owner = 'falcon', http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py b/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py index 2219de1..422f181 100644 --- a/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py +++ b/ambari-server/src/test/python/stacks/2.1/FALCON/test_falcon_server.py @@ -81,11 +81,15 @@ class TestFalconServer(RMFTestCase): def assert_configure_default(self): self.assertResourceCalled('Directory', '/var/run/falcon', owner = 'falcon', - recursive = True + recursive = True, + cd_access = "a", + mode = 0755, ) self.assertResourceCalled('Directory', '/var/log/falcon', owner = 'falcon', - recursive = True + recursive = True, + cd_access = "a", + mode = 0755, ) self.assertResourceCalled('Directory', '/var/lib/falcon/webapp', owner = 'falcon', @@ -243,10 +247,14 @@ class TestFalconServer(RMFTestCase): self.assertResourceCalled('Directory', '/var/run/falcon', owner = 'falcon', recursive = True, + cd_access = "a", + mode = 0755, ) self.assertResourceCalled('Directory', '/var/log/falcon', owner = 'falcon', recursive = True, + cd_access = "a", + mode = 0755, ) self.assertResourceCalled('Directory', '/usr/hdp/current/falcon-server/webapp', owner = 'falcon', http://git-wip-us.apache.org/repos/asf/ambari/blob/14c13c1c/ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py b/ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py index 4bf8e9d..df2a50f 100644 --- a/ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py +++ b/ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py @@ -42,27 +42,37 @@ class TestKnoxGateway(RMFTestCase): self.assertResourceCalled('Directory', '/usr/hdp/current/knox-server/data/', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/log/knox', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/run/knox', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/usr/hdp/current/knox-server/conf', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/usr/hdp/current/knox-server/conf/topologies', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('XmlConfig', 'gateway-site.xml', @@ -488,27 +498,37 @@ class TestKnoxGateway(RMFTestCase): self.assertResourceCalled('Directory', '/usr/hdp/current/knox-server/data/', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/log/knox', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/var/run/knox', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/usr/hdp/current/knox-server/conf', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('Directory', '/usr/hdp/current/knox-server/conf/topologies', owner = 'knox', group = 'knox', - recursive = True + recursive = True, + mode = 0755, + cd_access = "a", ) self.assertResourceCalled('XmlConfig', 'gateway-site.xml',
