-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63280/
-----------------------------------------------------------
(Updated Aug. 17, 2018, 5:54 p.m.)
Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Benjamin
Mahler, Greg Mann, and Jie Yu.
Changes
-------
Rebased.
Bugs: MESOS-8128
https://issues.apache.org/jira/browse/MESOS-8128
Repository: mesos
Description (updated)
-------
Since the containerizer launch depends on the inherited pipe to signal
the forked child, be explicit about manipulating O_CLOEXEC on the pipe
file descriptors. Make sure to close the pipe on the error paths.
Diffs (updated)
-----
src/slave/containerizer/mesos/containerizer.cpp
98129d006cda9b65804b518619b6addc8990410a
src/slave/containerizer/mesos/launcher.cpp
ed41a14892bdfa74d28bd0541ca188b86f8f96f8
src/slave/containerizer/mesos/linux_launcher.cpp
d1f8d3f56a9ef3aa98012c9395e74732e49c264d
Diff: https://reviews.apache.org/r/63280/diff/3/
Changes: https://reviews.apache.org/r/63280/diff/2-3/
Testing
-------
make check (Fedora 25)
Thanks,
James Peach
