----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40922/#review111765 -----------------------------------------------------------
Ship it! Master (e75cdba) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Dec. 23, 2015, 9:33 a.m., Martin Hrabovcin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40922/ > ----------------------------------------------------------- > > (Updated Dec. 23, 2015, 9:33 a.m.) > > > Review request for Aurora. > > > Bugs: AURORA-1548 > https://issues.apache.org/jira/browse/AURORA-1548 > > > Repository: aurora > > > Description > ------- > > This patch will provide way to **optionally** specify running process outputs > destination. Implementation was built on top of > https://reviews.apache.org/r/30695/ > > **What was changed:** > > New `destination` parameter is available on global cluster level and also on > each `Process` level. Possible options are `file` (default), `stream` to > parent process stdout/stderr, `mixed` will split output to files and stream > and finally `none` to discard any logs produced by running process. > > > Diffs > ----- > > NEWS ebfc3a6 > docs/configuration-reference.md cf63cfa > docs/deploying-aurora-scheduler.md 73b2e04 > src/main/python/apache/aurora/executor/bin/thermos_executor_main.py 7b7ef4b > src/main/python/apache/aurora/executor/thermos_task_runner.py 25fcca2 > src/main/python/apache/thermos/config/schema_base.py 5552108 > src/main/python/apache/thermos/core/process.py 8efdfdc > src/main/python/apache/thermos/core/runner.py 11c06a8 > src/main/python/apache/thermos/runner/thermos_runner.py a36bd2a > src/test/python/apache/thermos/core/test_process.py 261371d > > Diff: https://reviews.apache.org/r/40922/diff/ > > > Testing > ------- > > Unit test coverage is provided for new functionality. > > I did also manual testing with mesos/docker and I made sure that logs are > being written to expected files and also same output gets to docker daemon. > > > Thanks, > > Martin Hrabovcin > >