----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33318/#review80506 -----------------------------------------------------------
Patch looks great! Reviews applied: [33318] All tests passed. - Mesos ReviewBot On April 17, 2015, 6:21 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33318/ > ----------------------------------------------------------- > > (Updated April 17, 2015, 6:21 p.m.) > > > Review request for mesos, Benjamin Hindman, Bernd Mathiske, and Till > Toenshoff. > > > Repository: mesos > > > Description > ------- > > Fix docker containerizer usage and test. > The docker usage test is failing with the most recent change in including > executor resources in docker containerizer. > > > Diffs > ----- > > src/slave/containerizer/docker.hpp 6893684e6d199a5d69fc8bba8e60c4acaae9c3c9 > src/tests/docker_containerizer_tests.cpp > c772d4c836de18b0e87636cb42200356d24ec73d > > Diff: https://reviews.apache.org/r/33318/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy Chen > >