----------------------------------------------------------- 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