-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54038/
-----------------------------------------------------------
(Updated Nov. 25, 2016, 9:37 p.m.)
Review request for mesos, Adam B, Gastón Kleiman, Kapil Arya, and Joseph Wu.
Changes
-------
Fixed typos - thank for the review!
Bugs: MESOS-6396
https://issues.apache.org/jira/browse/MESOS-6396
Repository: mesos
Description
-------
For being able to supply environment additions for both, a task and
its executor separately we need to introduce a new hook as the
existing ones slavePreLaunchDockerHook (deprecated) as well as
slavePreLaunchDockerEnvironmentDecorator do not allow for this.
This new hook will likely allow for further additions like e.g.
adding volumes without having to adapt the signature but only the
returned proto message TaskExecutorDecoratorInfo.
Diffs (updated)
-----
include/mesos/hook.hpp f0606e3a68fa179cf7ea036f10563ef47c2aefa7
include/mesos/module/hook.proto PRE-CREATION
src/Makefile.am dd1626d177b38a6613f18f32bb0668abbb5100e0
src/hook/manager.hpp 5ecfcab48da808c84d36f9bcfcb5a8e0ad2167e5
src/hook/manager.cpp 24885226a788a7abd851e12b527f74fa972ec935
src/slave/containerizer/docker.cpp a8e522fc058f50560e8ec162c31be079e620bf9d
Diff: https://reviews.apache.org/r/54038/diff/
Testing
-------
make check
*WIP - functional test pending - unit tests pending - WIP*
Thanks,
Till Toenshoff