----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38910/#review101221 -----------------------------------------------------------
src/docker/docker.cpp (line 681) <https://reviews.apache.org/r/38910/#comment158593> Echo Jojy's comments, shall we make this as configurable? - Guangya Liu On 九月 30, 2015, 11:51 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38910/ > ----------------------------------------------------------- > > (Updated 九月 30, 2015, 11:51 p.m.) > > > Review request for mesos, Gilbert Song, Jojy Varghese, and Timothy Chen. > > > Bugs: MESOS-2613 > https://issues.apache.org/jira/browse/MESOS-2613 > > > Repository: mesos > > > Description > ------- > > Added `-v` flag to `docker rm`. > > > Diffs > ----- > > src/docker/docker.cpp afcedf1f1a309bd0626c33ee25694ac1b43bdec7 > > Diff: https://reviews.apache.org/r/38910/diff/ > > > Testing > ------- > > `make check` > > DockerContainerizerTest.ROOT_DOCKER_Launch_Executor fails on my Ubuntu 14.04 > VM, but this is a known issue: MESOS-3123 > > > Thanks, > > Greg Mann > >