----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24475/#review50485 -----------------------------------------------------------
Ship it! include/mesos/mesos.proto <https://reviews.apache.org/r/24475/#comment88305> s/an mesos/a Mesos/ src/docker/docker.cpp <https://reviews.apache.org/r/24475/#comment88306> s/a/an/ s/need/needs/ src/docker/docker.cpp <https://reviews.apache.org/r/24475/#comment88307> s/this but/but this/ src/docker/docker.cpp <https://reviews.apache.org/r/24475/#comment88309> s/to// src/slave/containerizer/docker.cpp <https://reviews.apache.org/r/24475/#comment88344> Sorry if I missed it, but how does logging redirection get resumed when the slave restarts? src/tests/docker_containerizer_tests.cpp <https://reviews.apache.org/r/24475/#comment88345> Why the change in sleep time? Why not "sleep 1000" which is used elsewhere for a "long" enough sleep. src/tests/docker_containerizer_tests.cpp <https://reviews.apache.org/r/24475/#comment88346> Ditto. src/tests/docker_containerizer_tests.cpp <https://reviews.apache.org/r/24475/#comment88347> Ditto. src/tests/docker_containerizer_tests.cpp <https://reviews.apache.org/r/24475/#comment88348> Ditto. I don't follow why there are different sleep times in these tests? - Ian Downes On Aug. 12, 2014, 10:47 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24475/ > ----------------------------------------------------------- > > (Updated Aug. 12, 2014, 10:47 p.m.) > > > Review request for mesos, Benjamin Hindman, Ian Downes, and Jie Yu. > > > Repository: mesos-git > > > Description > ------- > > Added new DockerInfo for future docker options, and allow command uris to be > fetched and mapped into docker before launching docker container. > > > Diffs > ----- > > include/mesos/mesos.proto cc9f20e > src/docker/docker.hpp 98b2d60 > src/docker/docker.cpp 1cba381 > src/slave/containerizer/containerizer.hpp 02754cd > src/slave/containerizer/containerizer.cpp c91ba38 > src/slave/containerizer/docker.cpp 904cdd3 > src/slave/containerizer/mesos/containerizer.cpp 694c9d1 > src/slave/flags.hpp 841de23 > src/slave/slave.cpp 787bd05 > src/tests/docker_containerizer_tests.cpp a559836 > src/tests/docker_tests.cpp 4ef1df4 > > Diff: https://reviews.apache.org/r/24475/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy Chen > >
