----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62800/#review197079 -----------------------------------------------------------
Patch looks great! Reviews applied: [62798, 62799, 62800] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Feb. 7, 2018, 8:01 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62800/ > ----------------------------------------------------------- > > (Updated Feb. 7, 2018, 8:01 p.m.) > > > Review request for mesos, James Peach, Joris Van Remoortere, and Joseph Wu. > > > Bugs: MESOS-7990 > https://issues.apache.org/jira/browse/MESOS-7990 > > > Repository: mesos > > > Description > ------- > > This patch added the support for systemd hierarchy in LinuxLauncher. > It created the same cgroup layout under the systemd hierarchy (if > systemd is enabled) as that in the freezer hierarchy. > > This can give us a bunch of benefits: > 1) systemd-cgls can list mesos container processes. > 2) systemd-cgtop can show stats for mesos containers. > 3) Avoid the pid migration issue described in MESOS-3352. > > For example: > > ``` > [jie@core-dev ~]$ systemd-cgls > |-1 /usr/lib/systemd/systemd --system --deserialize 20 > |-mesos > | |-8282b91a-5724-4964-a623-7c6bd68ff4ad > | |-31737 /usr/libexec/mesos/mesos-containerizer launch > | |-31739 mesos-default-executor --launcher_dir=/usr/libexec/mesos > | |-mesos > | |-8555f4af-fa4f-4c9c-aeb3-0c9f72e6a2de > | |-31791 /usr/libexec/mesos/mesos-containerizer launch > | |-31793 sleep 1000 > ``` > > > Diffs > ----- > > src/slave/containerizer/mesos/linux_launcher.cpp > c2e319894874e628beda6dc305462af0c274fd7b > > > Diff: https://reviews.apache.org/r/62800/diff/2/ > > > Testing > ------- > > sudo make check > > I also use the steps describe in MESOS-3352 to repo the systemd pid migration > issue. Didn't observe it after applying the patch. > > > Thanks, > > Jie Yu > >