-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56341/
-----------------------------------------------------------
(Updated Feb. 6, 2017, 3:19 p.m.)
Review request for mesos, Jie Yu and Kevin Klues.
Changes
-------
Changed approach, as the docker/runtime isolator is already full of
command-executor special-casing. This also doesn't break other isolators that
need to affect the command executor rather than the command task.
Summary (updated)
-----------------
Changed docker/runtime isolator's handling of Environment.
Bugs: MESOS-7027
https://issues.apache.org/jira/browse/MESOS-7027
Repository: mesos
Description (updated)
-------
This commit adds more special-casing in the `docker/runtime` isolator
for the command executor. The command executor will generally break
when the `docker/runtime` isolator provides environment variables
directly to the executor. This is because the environment variables
are provided in the context of the container image, rather than the
host.
For example, a container image may provide an environment variable
like `LD_LIBRARY_PATH=/image/specific/location`, whereas the default
executor expects to find libraries in the host's environment. If the
image's environment end up in the command executor at launch time,
the command executor may simply fail to launch.
Diffs (updated)
-----
src/launcher/executor.cpp 0c770bb18ae8bd8df85589b5262f457ab50574a9
src/slave/containerizer/mesos/isolators/docker/runtime.cpp
2d816e512c95ed2922c9578ba796908c5ce23da4
Diff: https://reviews.apache.org/r/56341/diff/
Testing (updated)
-------
In progress...
Thanks,
Joseph Wu
