mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jie Yu" <>
Subject Review Request 36930: Forced the network isolator to use the mount namespace.
Date Thu, 30 Jul 2015 00:19:22 GMT

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

Review request for mesos, Chi Zhang and Vinod Kone.

Repository: mesos


Forced the network isolator to use the mount namespace.

The code of the network isolator actually relies on the fact that the child is in a seprate
mount namespace. For example:

It originally depends on mount namespace, but was removed in this patch:

That was a bug to me. It didn't cause any issue because we don't clone the mounts (since we
are not using mount namespace) anymore after the above patch. So the kernel won't have an
extra reference to the mount when we try to umount it in `_cleanup()`.


  src/slave/containerizer/isolators/network/port_mapping.cpp 3f6e9df8711995d0dd3903c6170fdd5ad61aac5a



sudo make check


Jie Yu

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