Re: Review Request 41386: Logger Module: Add regression test for docker containerizer logging to sandbox.

2015-12-15 Thread Joseph Wu
> On Dec. 14, 2015, 8:08 p.m., Timothy Chen wrote: > > src/tests/container_logger_tests.cpp, line 158 > > > > > > Not against adding more tests, but we already have an identical test in > > the docker containerizer

Re: Review Request 41386: Logger Module: Add regression test for docker containerizer logging to sandbox.

2015-12-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41386/#review110488 --- Patch looks great! Reviews applied: [41002, 41003, 41004, 41061,

Review Request 41386: Logger Module: Add regression test for docker containerizer logging to sandbox.

2015-12-14 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41386/ --- Review request for mesos, Benjamin Hindman and Artem Harutyunyan. Bugs:

Re: Review Request 41386: Logger Module: Add regression test for docker containerizer logging to sandbox.

2015-12-14 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41386/#review110399 --- src/tests/container_logger_tests.cpp (line 158)