-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62215/
-----------------------------------------------------------
Review request for Ambari and Vitalyi Brodetskyi.
Bugs: AMBARI-21896
https://issues.apache.org/jira/browse/AMBARI-21896
Repository: ambari
Description
-------
Give this:
<property>
<name>dfs.datanode.data.dir.perm</name>
<value>750</value>
Ambari reports as changing permission from 750 to 755 on the data dirs:
2017-09-01 16:39:28,877 - Forcefully ensuring existence and permissions of the
directory: /hadoop01/data
2017-09-01 16:39:28,878 -
{'group': 'hadoop', 'cd_access': 'a', 'create_parents': True,
'ignore_failures': True, 'mode': 0755, 'owner': 'hdfs'}
2017-09-01 16:39:28,878 - Changing permission for /hadoop01/data from 750 to
755
2017-09-01 16:39:28,879 - Mount point for directory /hadoop02/data is
/hadoop02
2017-09-01 16:39:28,880 - Forcefully ensuring existence and permissions of the
directory: /hadoop02/data
2017-09-01 16:39:28,880 -
{'group': 'hadoop', 'cd_access': 'a', 'create_parents': True,
'ignore_failures': True, 'mode': 0755, 'owner': 'hdfs'}
2017-09-01 16:39:28,880 - Changing permission for /hadoop02/data from 750 to
755
2017-09-01 16:39:28,881 - Mount point for directory /hadoop03/data is
/hadoop03
2017-09-01 16:39:28,881 - Forcefully ensuring existence and permissions of the
directory: /hadoop03/data
2017-09-01 16:39:28,882 -
{'group': 'hadoop', 'cd_access': 'a', 'create_parents': True,
'ignore_failures': True, 'mode': 0755, 'owner': 'hdfs'}
2017-09-01 16:39:28,882 - Changing permission for /hadoop03/data from 750 to
755
2017-09-01 16:39:28,882 - Mount point for directory /hadoop04/data is
/hadoop04
Diffs
-----
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_datanode.py
1a54be0
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py
3c01059
ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 966254a
Diff: https://reviews.apache.org/r/62215/diff/1/
Testing
-------
mvn clean test
Thanks,
Andrew Onischuk