-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60280/
-----------------------------------------------------------
(Updated June 21, 2017, 10:26 p.m.)
Review request for mesos, Jie Yu, James Peach, and Zhitao Li.
Bugs: MESOS-7703
https://issues.apache.org/jira/browse/MESOS-7703
Repository: mesos
Description
-------
If a framework specifies use of its own executor and sets shell to false the executor is never
found. Additionally, the name of the binary is never passed as an argument so executors making
use of argv[0] will fail. This provides the full path to the executor so that the `execvp`
or `execvpe` is successful. The name of the binary is also passed as the first argument for
cases where there is no shell used.
Diffs (updated)
-----
src/slave/containerizer/mesos/launch.cpp 162ca1c2e
Diff: https://reviews.apache.org/r/60280/diff/2/
Changes: https://reviews.apache.org/r/60280/diff/1-2/
Testing
-------
`cd build && cmake .. -DCMAKE_BUILD_TYPE=Release && make -j4`
Also spun up a master and agent, connected and sent a task using the UCR (both with and without
the use of an OCI image) via our own framework, and checked the sandbox to verify that things
went accordingly.
Thanks,
Aaron Wood
|