Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/#review154819 --- Patch looks great! Reviews applied: [53378, 53407, 53406, 53385]

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/#review154787 --- Ship it! Ship It! - Alexander Rukletsov On Nov. 3, 2016,

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

> On Nov. 3, 2016, 3:57 p.m., Alexander Rukletsov wrote: > > src/tests/containerizer/docker_containerizer_tests.cpp, line 3921 > > > > > > I'm surprised to find out that we don't have `DockerExecutorTest` > >

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/ --- (Updated Nov. 3, 2016, 4:58 p.m.) Review request for mesos, Alexander

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/ --- (Updated Nov. 3, 2016, 4:49 p.m.) Review request for mesos, Alexander

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/ --- (Updated Nov. 3, 2016, 4:08 p.m.) Review request for mesos, Alexander

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/ --- (Updated Nov. 3, 2016, 4:05 p.m.) Review request for mesos, Alexander

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/#review154713 --- src/tests/containerizer/docker_containerizer_tests.cpp (lines

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/ --- (Updated Nov. 3, 2016, 3:52 p.m.) Review request for mesos, Alexander

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/#review154679 --- Patch looks great! Reviews applied: [53378, 53406, 53407, 53385]

Re: Review Request 53385: Added a test to ensure MESOS-6457 is fixed for the Docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53385/ --- (Updated Nov. 2, 2016, 10:06 p.m.) Review request for mesos, Alexander