This is an automatically generated e-mail. To reply, visit:

(Updated Jan. 4, 2016, 12:25 p.m.)

Review request for Aurora and John Sirois.


Updated patch addresses @StephanErb comments. @ReviewBot retry

Bugs: AURORA-1548

Repository: aurora


This patch will provide way to **optionally** specify running process outputs 
destination. Implementation was built on top of 

**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 (updated)

  NEWS c0c454d 
  docs/configuration-reference.md cf63cfa 
  docs/deploying-aurora-scheduler.md c0988e8 
  src/main/python/apache/aurora/executor/bin/thermos_executor_main.py 4e9b027 
  src/main/python/apache/aurora/executor/thermos_task_runner.py c019fc9 
  src/main/python/apache/thermos/config/schema_base.py 5552108 
  src/main/python/apache/thermos/core/process.py 8a181b0 
  src/main/python/apache/thermos/core/runner.py 5623dce 
  src/main/python/apache/thermos/runner/thermos_runner.py 3dacd45 
  src/test/python/apache/thermos/core/test_process.py da4c494 

Diff: https://reviews.apache.org/r/40922/diff/


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.


Martin Hrabovcin

Reply via email to