mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Peach <>
Subject Re: Review Request 69132: Used mount(8) to make XFS loop device mounts.
Date Wed, 24 Oct 2018 15:41:48 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 24, 2018, 3:41 p.m.)

Review request for mesos, Ilya Pronin, Jacob Janco, and Jiang Yan Xu.

Repository: mesos


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

Diffs (updated)

  src/tests/containerizer/xfs_quota_tests.cpp 084df067a8d97efdc86fc0ed878bc3f2edca06e5 




sudo make check (Fedora 28, CentOS 6)


James Peach

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message