-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37612/
-----------------------------------------------------------
Review request for Ambari, Myroslav Papirkovskyy and Vitalyi Brodetskyi.
Bugs: AMBARI-12817
https://issues.apache.org/jira/browse/AMBARI-12817
Repository: ambari
Description
-------
Permissions on the container-executor need to be set so the group used for the
hadoop components but if YARN is installed after-the-fact (i.e. install
cluster, then add service YARN) and you are using a non-default group (like
"hadooptest") they are not being set.
> ls -la /usr/hdp/current/hadoop-yarn-nodemanager/bin/container-executor
---Sr-s--- 1 root hadoop 115062 Jul 14 09:07
/usr/hdp/current/hadoop-yarn-nodemanager/bin/container-executor
Should be:
# ls -la /usr/hdp/current/hadoop-yarn-nodemanager/bin/container-executor
---Sr-s--- 1 root hadooptest 115062 Jul 14 09:07
/usr/hdp/current/hadoop-yarn-nodemanager/bin/container-executor
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackLevelConfigurationResourceDefinition.java
ce6a5b3
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
4afa9b0
ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java
4b88aff
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration/yarn-site.xml
686e40c
ambari-server/src/main/resources/key_properties.json ff20b8b
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py
3fd010a
ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/configuration/yarn-site.xml
049eeb2
ambari-server/src/test/python/stacks/2.0.6/common/test_stack_advisor.py
b478cc0
ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py e764739
ambari-web/app/controllers/wizard/step7_controller.js 4d13b5b
ambari-web/app/mixins/common/serverValidator.js b0c0772
Diff: https://reviews.apache.org/r/37612/diff/
Testing
-------
In progress
Thanks,
Dmytro Sen
