mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 51930: Introduced Linux capabilities support for Mesos executor.
Date Tue, 20 Sep 2016 13:42:16 GMT


> On Sept. 20, 2016, 2:36 a.m., Jie Yu wrote:
> > src/launcher/executor.cpp, line 817
> > <https://reviews.apache.org/r/51930/diff/1/?file=1502557#file1502557line817>
> >
> >     It looks wierd that we have ifdef here but not have that for the field member.
I'd suggest we don't do any ifdef for flags as they are optional anyway. Otherwise, you'll
have to change all the constructors as well, which is a little messy.

Hmm, that would mean that that flag would be there under e.g., macos or windows, but have
no effect which we'd need to patch by either documentation or adding a validation step.

I am not sure I understand your comment re:constructors; the member is there and can be default
constructed. I think the code should be fine as is.


- Benjamin


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


On Sept. 19, 2016, 4:19 p.m., Benjamin Bannier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51930/
> -----------------------------------------------------------
> 
> (Updated Sept. 19, 2016, 4:19 p.m.)
> 
> 
> Review request for mesos and Jie Yu.
> 
> 
> Bugs: MESOS-5275
>     https://issues.apache.org/jira/browse/MESOS-5275
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This change introduces Linux capability-based security the Mesos
> exector. A new flag `capabilities` is introduced to optionally specify
> the capabilities tasks launched by the Mesos executor are allowed to
> use.
> 
> 
> Diffs
> -----
> 
>   src/launcher/executor.cpp 5370634ef9e6f3ac9717fed71f6a77707026a16a 
>   src/launcher/posix/executor.hpp 9e467263dcc0623fce83b18887896547db9aa16b 
>   src/launcher/posix/executor.cpp 7c40ebebb983b7977526fee123bf07c86fbb5e50 
> 
> Diff: https://reviews.apache.org/r/51930/diff/
> 
> 
> Testing
> -------
> 
> Tested as part of https://reviews.apache.org/r/50271/ on ubuntu-14.04 w/ gcc-4.8.4 w/o
optimizations.
> 
> 
> Thanks,
> 
> Benjamin Bannier
> 
>


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