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

(Updated Dec. 23, 2015, 9:33 a.m.)

Review request for Aurora.


Fixed small bug after manual testing and minor docs changes. @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 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/


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