----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69132/#review209975 -----------------------------------------------------------
PASS: Mesos patch 69132 was successfully built and tested. Reviews applied: `['69132']` All the build artifacts available at: http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2520/mesos-review-69132 - Mesos Reviewbot Windows On Oct. 24, 2018, 3:41 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69132/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2018, 3:41 p.m.) > > > Review request for mesos, Ilya Pronin, Jacob Janco, and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > When we added the `filesystem/linux` isolator to the XFS isolation > tests, we broke them for older Linux distributions (specifically > CentOS 6). The `filesystem/linux` isolator needs to remount the > work directory in shared mode using `mount(8)`, but `mount(8)` > requires the mount point it is manipulating to be present in > `mtab(5)`. Since the XFS tests simply issued a `mount(2)` system > call, `mtab(5)` wasn't updated and the remount would fail. Modern > Linux distributions don't see this problem because `/` is mounted > shared by default, and `mtab(5)` is a symlink to `/proc/self/mounts` > (i.e. always in sync). > > The straightforward fix is to run `mount(8)` to make the XFS test > fixture mounts, so that `mount(8)` always sees a consistent view of > `mtab(5)`. > > > Diffs > ----- > > src/tests/containerizer/xfs_quota_tests.cpp > 084df067a8d97efdc86fc0ed878bc3f2edca06e5 > > > Diff: https://reviews.apache.org/r/69132/diff/2/ > > > Testing > ------- > > sudo make check (Fedora 28, CentOS 6) > > > Thanks, > > James Peach > >
