----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65049/#review195072 -----------------------------------------------------------
PASS: Mesos patch 65049 was successfully built and tested. Reviews applied: `['65049']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/65049 - Mesos Reviewbot Windows On Jan. 9, 2018, 7:31 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65049/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2018, 7:31 p.m.) > > > Review request for mesos, Gilbert Song, James Peach, and Jiang Yan Xu. > > > Bugs: MESOS-8356 > https://issues.apache.org/jira/browse/MESOS-8356 > > > Repository: mesos > > > Description > ------- > > This patch fixes MESOS-8356 by skipping the current container to be > launched when doing the shared volume check (`isVolumeInUse`). Prior to > this patch, the code is buggy because `isVolumeInUse` will always be set > to `true`. > > > Diffs > ----- > > src/slave/containerizer/docker.cpp 890cb2aba44fe76e891117833eac8ccca00b759b > > > Diff: https://reviews.apache.org/r/65049/diff/1/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Jie Yu > >