mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "R.B. Boyer" <boyers...@nexusvector.net>
Subject Re: Review Request 26622: Command Executor is broken when used with shell=false
Date Fri, 24 Oct 2014 18:15:01 GMT


> On Oct. 24, 2014, 11:40 a.m., Timothy Chen wrote:
> > src/tests/slave_tests.cpp, line 448
> > <https://reviews.apache.org/r/26622/diff/4/?file=731893#file731893line448>
> >
> >     Just tried bin/ls on OSX and realize Mac's ls has no --author flag so this is
going to fail on OSX.
> >     
> >     It's ok if we don't support it but then we need to wrap the test in #ifdef __linux__

The part that actually breaks is any double-dash option arguments.  If the OSX version of
ls shares any other double-dash arg options with linux then just replace --author with that.


- R.B.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26622/#review58287
-----------------------------------------------------------


On Oct. 23, 2014, 9:04 p.m., R.B. Boyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26622/
> -----------------------------------------------------------
> 
> (Updated Oct. 23, 2014, 9:04 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Jie Yu, and Timothy Chen.
> 
> 
> Bugs: MESOS-1873
>     https://issues.apache.org/jira/browse/MESOS-1873
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Basically if you use "shell=false" with a non-empty argument list and the Command Executor
it is completely broken.
> 
> When we clone the env vars to fork "mesos-executor" all of the original cmd args are
cloned as well (unintentionally) due to some protocol-buffer merge shenanigans.
> 
> Don't pass task-related arguments to mesos-executor.
> 
> The description on the linked jira ticket goes into more detail.
> 
> 
> Diffs
> -----
> 
>   src/slave/slave.cpp 55e5264 
>   src/tests/slave_tests.cpp a1bd00c 
> 
> Diff: https://reviews.apache.org/r/26622/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> added new test "SlaveTest.GetExecutorInfo" verifying the explicit desired behavior.
> 
> 
> Thanks,
> 
> R.B. Boyer
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message