-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61112/#review181366
-----------------------------------------------------------
src/examples/balloon_framework.cpp
Lines 120-124 (patched)
<https://reviews.apache.org/r/61112/#comment256905>
I would suggest to use a different approach here -- instead of allowing multiple URIs
to be used for the executor artefact, lets simply go for using archives in those cases.
Now the issue preventing us to do so here is https://github.com/apache/mesos/blob/cd3380c4e9521b4b26f9030658816eee7a4b89a1/src/examples/balloon_framework.cpp#L493
Hence my suggestion is to add a new flag called e.g. `executor_executable` which can be
used if we want the fetcher to make non archive downloads `chmod` into `+X`. Whenever that
flag is not set, we can now allow archive downloads and achieve what we want without having
to;
- trigger multiple downloads
- accept depending libraries to be `chmod` into `+X` which wont trigger failures but still
is nasty
- Till Toenshoff
On July 25, 2017, 3:48 p.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61112/
> -----------------------------------------------------------
>
> (Updated July 25, 2017, 3:48 p.m.)
>
>
> Review request for mesos, Alexander Rukletsov and Alexander Rojas.
>
>
> Bugs: MESOS-7814
> https://issues.apache.org/jira/browse/MESOS-7814
>
>
> Repository: mesos
>
>
> Description
> -------
>
> This allows to set URIs that will be fetched and executed before
> running the executor.
>
>
> Diffs
> -----
>
> src/examples/balloon_framework.cpp b8c8e387bd8592bc1d27ff62fa9fd5397ad71fb9
>
>
> Diff: https://reviews.apache.org/r/61112/diff/1/
>
>
> Testing
> -------
>
> $ make check
>
>
> Thanks,
>
> Armand Grillet
>
>
|