mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrei Budnik <>
Subject Review Request 68257: Fixed incorrect `mnt` namespace detection of command executor's task.
Date Tue, 07 Aug 2018 13:46:53 GMT

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

Review request for mesos, Alexander Rukletsov, Gilbert Song, Jie Yu, and Kevin Klues.

Bugs: MESOS-9116

Repository: mesos


Previously, we were walking the process tree from the container's
`init` process to find the first process along the way whose `mnt`
namespace differs from the `init` process. We expected this algorithm
to always return the PID of the command executor's task. However, if
someone launches multiple nested containers within the process tree,
the algorithm might detect the PID of the nested container instead of
the command executor's task. The detected PID might belong to a
short-lived container, so the container's process might terminate at
the moment the containerizer launcher (aka `nanny`) process tries to
enter its `mnt` namespace. This patch fixes the detection algorithm
so that it always returns PID of the command executor's task.


  src/slave/containerizer/mesos/utils.cpp 30e76d1d91651975033078f5450e45f5f2fd8ba0 



1) Internal CI with disabled `ROOT_CGROUPS_LaunchNestedContainerSessionsInParallel` test (see
previous patch).
2) Fedora 25: `./src/mesos-tests --gtest_filter=*AgentAPITest.LaunchNestedContainerSessionInParallel*
--gtest_break_on_failure --gtest_repeat=100 --verbose`


Andrei Budnik

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