----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31766/ -----------------------------------------------------------
(Updated March 5, 2015, 3:34 p.m.) Review request for Ambari, Dmitro Lisnichenko and Vitalyi Brodetskyi. Bugs: AMBARI-9943 https://issues.apache.org/jira/browse/AMBARI-9943 Repository: ambari Description ------- This is double permission restriction which we seems like haven't yet tested. The most common problem I see over the code that hardcore umask removes some x bits. We didn't really need them until we were using root account, but now it becomes a problem in multiple or more places. Diffs (updated) ----- ambari-agent/conf/unix/ambari-env.sh bf7ca55 ambari-agent/src/test/python/resource_management/TestDirectoryResource.py fe64400 ambari-agent/src/test/python/resource_management/TestExecuteResource.py 216fd1a ambari-agent/src/test/python/resource_management/TestFileResource.py 28fa610 ambari-agent/src/test/python/resource_management/TestLinkResource.py cdb6061 ambari-agent/src/test/python/resource_management/TestPropertiesFileResource.py bdb64de ambari-agent/src/test/python/resource_management/TestXmlConfigResource.py 4affd31 ambari-common/src/main/python/resource_management/core/providers/system.py ac63b21 ambari-common/src/main/python/resource_management/core/sudo.py ae21f84 ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive.py 64bcebc ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py 745402a ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_metastore.py 9153a84 ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py 5230196 ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py e038ddf ambari-server/src/test/python/stacks/2.1/HIVE/test_hive_metastore.py 990eac8 Diff: https://reviews.apache.org/r/31766/diff/ Testing ------- mvn clean test Thanks, Andrew Onischuk