----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63829/#review195648 -----------------------------------------------------------
PASS: Mesos patch 63829 was successfully built and tested. Reviews applied: `['63829']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/63829 - Mesos Reviewbot Windows On Jan. 17, 2018, 11:24 a.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63829/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2018, 11:24 a.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, and Zhitao Li. > > > Repository: mesos > > > Description > ------- > > This includes two changes: > 1) Containerizer::remove no longer CHECKs for a nested container as > input and accepts standalone containers too. > 2) Standalone containers no longer have their runtime directory > deleted upon exit (which funnily, would prevent the containerizer > from figuring out that the ContainerID is a standalone). > > This change means that, for standalone containers, the caller will > need to explicitly remove the standalone container with the > REMOVE_CONTAINER API. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.cpp > ec39d04e7129a4f8b87166f18d3203dc280c373b > src/tests/agent_container_api_tests.cpp > 618569277545205017320aaf1f3a70e540d35e30 > > > Diff: https://reviews.apache.org/r/63829/diff/2/ > > > Testing > ------- > > make check > > > Thanks, > > Joseph Wu > >