-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54103/
-----------------------------------------------------------
Review request for mesos, Gilbert Song and Kevin Klues.
Bugs: MESOS-6648
https://issues.apache.org/jira/browse/MESOS-6648
Repository: mesos
Description
-------
Prior to this patch, the launch helper takes various flags from
MesosContainerizer to launch the container. This makes it very hard to
add more parameters to the launch helper. This patch simplifies this
by passing 'ContainerLaunchInfo' instead. 'ContainerLaunchInfo' is
also the protobuf message returned by isolators during 'prepare()'.
This makes it very easy to merge them and send it to the launch
helper. More importantly, this makes it very easy to add more
parameters to the launch helper in the future.
Diffs
-----
include/mesos/slave/containerizer.proto 2621522ae59cf9275f607679b4678ac54508993d
src/launcher/posix/executor.cpp da0081c0e470aebb16d2e78031d276f5d7d2c726
src/slave/containerizer/mesos/containerizer.cpp 9b33495d9babc3ee489a8712fe1977746c41043f
src/slave/containerizer/mesos/isolators/docker/volume/isolator.cpp 6f0d6b0c9c1d78e5fecbbef4c6ff03825356799d
src/slave/containerizer/mesos/isolators/filesystem/linux.cpp 8f024d084189b59bb229c63d20108e7bfe42065f
src/slave/containerizer/mesos/isolators/filesystem/shared.cpp d16b42f2231ba2e7779e798efe05e2bbd20dfac9
src/slave/containerizer/mesos/isolators/namespaces/pid.cpp 5bf5ffbceed1229fad35ca94f42b93c097152fe5
src/slave/containerizer/mesos/isolators/network/cni/cni.cpp 3b2d4db325b23a13fa0c1a4035f309816de181fd
src/slave/containerizer/mesos/isolators/network/port_mapping.cpp a994adf78898b0e55ced8a2214287a97edc16d38
src/slave/containerizer/mesos/launch.hpp 155e3c5a27b8c710971ee4b508600d3b5589a2e0
src/slave/containerizer/mesos/launch.cpp 320e42748adbabf09f77cb4f5951e2a7ea58fe64
src/tests/containerizer/port_mapping_tests.cpp 79b0a60c48a81c2f868d361ca07373bf3c2a8477
Diff: https://reviews.apache.org/r/54103/diff/
Testing
-------
sudo make check
Thanks,
Jie Yu
|