-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60280/#review179735
-----------------------------------------------------------
Bad patch!
Reviews applied: [60280]
Failed command: python support/apply-reviews.py -n -r 60280
Error:
Traceback (most recent call last):
File "support/apply-reviews.py", line 417, in <module>
main()
File "support/apply-reviews.py", line 412, in main
reviewboard(options)
File "support/apply-reviews.py", line 402, in reviewboard
apply_review(options)
File "support/apply-reviews.py", line 158, in apply_review
fetch_patch(options)
File "support/apply-reviews.py", line 215, in fetch_patch
shell(cmd, options['dry_run'])
UnboundLocalError: local variable 'cmd' referenced before assignment
Full log: http://mesos-winbot.westus.cloudapp.azure.com/logs/125/console
- Mesos Reviewbot Windows
On July 5, 2017, 9:03 p.m., Aaron Wood wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60280/
> -----------------------------------------------------------
>
> (Updated July 5, 2017, 9:03 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. This provides the full path to the executor so that
> the `execvp` or `execvpe` is successful.
>
>
> Diffs
> -----
>
> src/slave/containerizer/mesos/launch.cpp 162ca1c2e
>
>
> Diff: https://reviews.apache.org/r/60280/diff/7/
>
>
> 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
>
>
