-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48428/
-----------------------------------------------------------
Review request for mesos and Jie Yu.
Bugs: MESOS-4279
https://issues.apache.org/jira/browse/MESOS-4279
Repository: mesos
Description
-------
Previously, Docker::run would return a Future<Nothing> which
requires the caller to perform an inspect to obtain the exit
status of the container. This is difficult to get right, and
so this patch leverages the fact that 'docker run' already
returns the exit status from the container termination to
simplify the calling code. Namely, in the docker executor we
need to use the exit status to determine how to send a
terminal status update (see the command executor code).
Diffs
-----
src/docker/docker.hpp d9ffc18496718701fac8182506a8c36e21e9c319
src/docker/docker.cpp ffbd7ef56d02dfe550a60851251e6645d2ca5925
src/docker/executor.cpp 1b3a7795b1db83394d4b884c1041c341f88a7df1
src/slave/containerizer/docker.cpp 63efbe6f45958d44d60fe4a7fea816f5fb0457b2
src/tests/containerizer/docker_containerizer_tests.cpp
5591a6784afd10b4c7535f90c2e6745fa74c318b
src/tests/containerizer/docker_tests.cpp
7ef52ade0d3389f9e24e3c5c7dda4f8809b9d83f
src/tests/mesos.hpp 6f2888023c957f0af4f7374c98e406816a8423e3
Diff: https://reviews.apache.org/r/48428/diff/
Testing
-------
sudo make check
Thanks,
Benjamin Mahler
