Re: Review Request 32798: Add test to verify executor clean up in docker containerizer.

2015-04-03 Thread Timothy Chen
> On April 3, 2015, 3:38 p.m., Benjamin Hindman wrote: > > src/tests/docker_containerizer_tests.cpp, lines 2625-2627 > > > > > > My only question here is how do you know the executor is properly > > killed and cleane

Re: Review Request 32798: Add test to verify executor clean up in docker containerizer.

2015-04-03 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32798/#review78783 --- Ship it! src/tests/docker_containerizer_tests.cpp

Re: Review Request 32798: Add test to verify executor clean up in docker containerizer.

2015-04-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32798/#review78749 --- Patch looks great! Reviews applied: [32796, 32797, 32798] All test

Review Request 32798: Add test to verify executor clean up in docker containerizer.

2015-04-02 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32798/ --- Review request for mesos, Benjamin Hindman, Bernd Mathiske, and Till Toenshoff.