-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49935/
-----------------------------------------------------------
(Updated 7月 25, 2016, 10:55 a.m.)
Review request for Ambari, Andrew Onischuk, jun aoki, and Myroslav Papirkovskyy.
Bugs: AMBARI-17633
https://issues.apache.org/jira/browse/AMBARI-17633
Repository: ambari
Description
-------
When installing YARN+MapReduce2, I got a WARN message in nodemanager log(in Log
Search View) like below
```
2016-07-09 06:30:21,865 WARN logaggregation.LogAggregationService
LogAggregationService.java:230 - Remote Root Log Dir [/app-logs] already exist,
but with incorrect permissions. Expected: [rwxrwxrwt], Found: [rwxrwxrwx]. The
cluster may have problems with multiple users.
```
I think the cause of this WARN is
[this](https://github.com/apache/ambari/blob/trunk/ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py#L115).
We should add stickybit to yarn.nodemanager.remote-app-log-dir.
Diffs
-----
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py
cf5736e
ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py 4600063
Diff: https://reviews.apache.org/r/49935/diff/
Testing (updated)
-------
mvn clean test && manual test
+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12817505/AMBARI-17633.2.patch
against trunk revision .
+1 @author. The patch does not contain any @author tags.
+1 tests included. The patch appears to include 1 new or modified test files.
+1 javac. The applied patch does not increase the total number of javac
compiler warnings.
+1 release audit. The applied patch does not increase the total number of
release audit warnings.
+1 core tests. The patch passed unit tests in ambari-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/7805//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/7805//console
This message is automatically generated.
Thanks,
Masahiro Tanaka
