Re: Review Request 52682: Cleaned up a few style issues in the capabilities isolator tests.

> On Oct. 18, 2016, 12:02 a.m., Jie Yu wrote: > > src/tests/containerizer/linux_capabilities_isolator_tests.cpp, line 100 > > > > > > const UsaImage& I believe we always pass fundamental types like this enum by valu

Re: Review Request 52682: Cleaned up a few style issues in the capabilities isolator tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52682/ --- (Updated Oct. 18, 2016, 8:59 a.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 51033: Added validation benchmark test for resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51033/ --- (Updated 十月 18, 2016, 7:05 a.m.) Review request for mesos, Benjamin Mahler and

Re: Review Request 51493: Add the query parameter in createRequest.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51493/#review153039 --- Handle the error, and then we'll consider this an appropriate repl

Re: Review Request 51495: Replace http::get with http::request in libprocess project.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51495/#review153041 --- Looks good besides the wrapping 3rdparty/libprocess/src/tests/ht

Re: Review Request 51505: Replace http::get with http::request in mesos project.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51505/#review153044 --- src/tests/executor_http_api_tests.cpp (lines 310 - 312)

Re: Review Request 52741: Added capabilities support to mesos-execute.

> On Oct. 18, 2016, 12:10 a.m., Jie Yu wrote: > > src/cli/execute.cpp, line 690 > > > > > > Why the extra `()`? Misguided consistency with the check above :/ ... I reordered and reformated the condition. - Benjam

Re: Review Request 52741: Added capabilities support to mesos-execute.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52741/ --- (Updated Oct. 18, 2016, 9:07 a.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 52967: Improved documentation for shared persistent volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52967/#review153036 --- Fix it, then Ship it! docs/home.md (line 54)

Re: Review Request 52780: Added input and output functions for v1::CapabilityInfo.

> On Oct. 18, 2016, 12:11 a.m., Jie Yu wrote: > > src/v1/mesos.cpp, lines 517-533 > > > > > > Can you follow the same impl in v0? In fact I would have liked to use the same tooling v0 uses here, but could find `JSO

Re: Review Request 46227: Added an user to indicate owner of persistent volume.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46227/ --- (Updated Oct. 18, 2016, 7:42 a.m.) Review request for mesos and Jiang Yan Xu.

Review Request 52980: Updated os::mkdir() to take the mode as an input.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52980/ --- Review request for mesos and Jiang Yan Xu. Bugs: MESOS-4893 https://issues.

Re: Review Request 46228: Create persistent volume with a supplied user.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46228/ --- (Updated Oct. 18, 2016, 7:45 a.m.) Review request for mesos and Jiang Yan Xu.

Re: Review Request 46229: Add unit tests for adding a user for persistent volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46229/ --- (Updated Oct. 18, 2016, 7:46 a.m.) Review request for mesos and Jiang Yan Xu.

Re: Review Request 46230: Updated docs to reflect user in persistent volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46230/ --- (Updated Oct. 18, 2016, 7:46 a.m.) Review request for mesos and Jiang Yan Xu.

Re: Review Request 52783: Added documentation for mesos-containerizer Linux capabilities support.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52783/ --- (Updated Oct. 18, 2016, 9:51 a.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 52783: Added documentation for mesos-containerizer Linux capabilities support.

> On Oct. 18, 2016, 6:55 a.m., Avinash sridharan wrote: > > docs/linux_capabilities.md, line 3 > > > > > > s/described/describes > > > > to add support for `Linux Capabilities` I do not understand your second

Re: Review Request 46824: Fully qualified addresses of Flag members in add calls.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46824/#review153049 --- src/local/main.cpp (lines 48 - 63)

Re: Review Request 52624: Replaced POSIX `int` with `FileDesc` abstraction in `src` folder.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52624/#review153052 --- Bad patch! Reviews applied: [52624, 52625, 52972, 52544, 52364, 5

Re: Review Request 52390: Fully qualified addresses of Flag members in add calls in stout.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52390/#review153053 --- Fix it, then Ship it! 3rdparty/stout/tests/flags_tests.cpp (li

Re: Review Request 46824: Fully qualified addresses of Flag members in add calls.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46824/ --- (Updated Oct. 18, 2016, 10:39 a.m.) Review request for mesos, Alexander Ruklets

Re: Review Request 46824: Fully qualified addresses of Flag members in add calls.

> On Oct. 18, 2016, 9:54 a.m., Michael Park wrote: > > src/local/main.cpp, lines 48-63 > > > > > > Is there a reason why we don't just add this directly to `local::Flags`? `local::Flags` is used in some other places

Re: Review Request 52682: Cleaned up a few style issues in the capabilities isolator tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52682/ --- (Updated Oct. 18, 2016, 10:48 a.m.) Review request for mesos and Jie Yu. Chan

Re: Review Request 46824: Fully qualified addresses of Flag members in add calls.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46824/#review153055 --- Fix it, then Ship it! I'll just make the adjustments below when

Re: Review Request 46824: Fully qualified addresses of Flag members in add calls.

> On Oct. 18, 2016, 11:22 a.m., Michael Park wrote: > > I'll just make the adjustments below when I commit it. Please let me know > > if you're against any of them. Thanks for the careful review. Your suggested adjustments are needed, +1. - Benjamin -

Re: Review Request 52741: Added capabilities support to mesos-execute.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52741/#review153065 --- Patch looks great! Reviews applied: [52780, 52741] Passed comman

Re: Review Request 51785: [HBASE-16608] Merge for the segments in the compaction pipeline and simplifying the user interface for in-memory flush

> On Oct. 16, 2016, 5:13 p.m., Anoop Sam John wrote: > > hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HeapMemStoreLAB.java, > > line 179 > > > > > > Able to get what u r trying to do here. Seem

Re: Review Request 51785: [HBASE-16608] Merge for the segments in the compaction pipeline and simplifying the user interface for in-memory flush

> On Oct. 16, 2016, 10:43 p.m., Anoop Sam John wrote: > > hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HeapMemStoreLAB.java, > > line 179 > > > > > > Able to get what u r trying to do here. See

Re: Review Request 46230: Updated docs to reflect user in persistent volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46230/#review153073 --- Patch looks great! Reviews applied: [46227, 52980, 46228, 46229,

Review Request 52985: Updated CHANGELOG in preparation for RC1 of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52985/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos Descripti

Review Request 52986: Updated endpoint-help in preparation of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52986/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos Descripti

Re: Review Request 51785: [HBASE-16608] Merge for the segments in the compaction pipeline and simplifying the user interface for in-memory flush

> On Oct. 16, 2016, 10:43 p.m., Anoop Sam John wrote: > > hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HeapMemStoreLAB.java, > > line 179 > > > > > > Able to get what u r trying to do here. See

Re: Review Request 52682: Cleaned up a few style issues in the capabilities isolator tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52682/#review153081 --- Patch looks great! Reviews applied: [52682] Passed command: expo

Re: Review Request 51052: Make mesos-docker-execute understand cgroups_enable_cfs.

> On Oct. 18, 2016, 4:49 a.m., Jie Yu wrote: > > src/docker/docker.hpp, line 51 > > > > > > Having this as a global knob sounds really weird to me. I know you want > > to work around the Docker::run GMock issue. But

Review Request 52988: Fixed a typo in a comment.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52988/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos Descripti

Re: Review Request 50380: Added new benchmark test for port resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50380/ --- (Updated 十月 18, 2016, 3:02 p.m.) Review request for mesos, Benjamin Mahler, Kla

Re: Review Request 52985: Updated CHANGELOG in preparation for RC1 of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52985/#review153094 --- Fix it, then Ship it! CHANGELOG (lines 1 - 2)

Re: Review Request 52986: Updated endpoint-help in preparation of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52986/ --- (Updated Oct. 18, 2016, 3:14 p.m.) Review request for mesos and Alexander Rukle

Re: Review Request 52986: Updated endpoint-help in preparation of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52986/#review153096 --- Ship it! I assume this is autoimatically generated *without* ma

Re: Review Request 52985: Updated CHANGELOG in preparation for RC1 of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52985/ --- (Updated Oct. 18, 2016, 3:24 p.m.) Review request for mesos and Alexander Rukle

Re: Review Request 52783: Added documentation for mesos-containerizer Linux capabilities support.

> On Oct. 18, 2016, 4:55 a.m., Avinash sridharan wrote: > > docs/configuration.md, line 1003 > > > > > > s/the agent/that the agent/ > > Benjamin Bannier wrote: > Not a native speaker but to me this seems to ju

Re: Review Request 52986: Updated endpoint-help in preparation of 1.1.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52986/#review153101 --- Patch looks great! Reviews applied: [52985, 52986] Passed comman

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

> On Oct. 18, 2016, 3:14 a.m., James Peach wrote: > > 3rdparty/libprocess/src/encoder.hpp, line 291 > > > > > > It's not obvious to me that this is the right change since the > > surrounding code tries to deal with

Re: Review Request 51027: Track allocation candidates to bound allocator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51027/ --- (Updated Oct. 18, 2016, 4:14 p.m.) Review request for mesos, Benjamin Mahler, G

Re: Review Request 52967: Improved documentation for shared persistent volumes.

> On Oct. 18, 2016, 7:19 a.m., Jiang Yan Xu wrote: > > docs/home.md, line 54 > > > > > > s/tasks/tasks across containers/ > > > > as multiple tasks sharing an executor today can already access the same > >

Re: Review Request 52967: Improved documentation for shared persistent volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52967/ --- (Updated Oct. 18, 2016, 4:15 p.m.) Review request for mesos and Jiang Yan Xu.

Re: Review Request 52939: Updated CLI pylint configuration to allow 2 character variable names.

> On Oct. 18, 2016, 1 a.m., Joseph Wu wrote: > > src/cli_new/pylint.config, line 13 > > > > > > I see a single 2 character variable name `io` here: > > https://reviews.apache.org/r/52953/diff/1#0 -- Line 306 > >

Re: Review Request 52940: Updated CLI pylint configuration to allow up to 30 local variables.

> On Oct. 18, 2016, 1 a.m., Joseph Wu wrote: > > src/cli_new/pylint.config, line 14 > > > > > > Looks like `__enter_namespaces` in https://reviews.apache.org/r/52953/ > > is over the default limit of 15 by one variab

Re: Review Request 52941: Updated CLI pylint configuration to allow 0 public methods.

> On Oct. 18, 2016, 1 a.m., Joseph Wu wrote: > > src/cli_new/pylint.config, line 15 > > > > > > Is this used anywhere? I did not encounter a linting error for this in > > your chain. I know I encountered it at some

Re: Review Request 51008: Added infrastructure for unit tests in the new python-based CLI.

> On Oct. 18, 2016, 1:01 a.m., Joseph Wu wrote: > > src/cli_new/bin/tests.py, lines 23-25 > > > > > > This actually failed to lint after applying: > > ``` > > Checking 1 Python file > > * Modul

Re: Review Request 52953: Added container 'exec' command to the CLI.

> On Oct. 18, 2016, 1:01 a.m., Joseph Wu wrote: > > src/cli_new/lib/mesos/plugins/container/main.py, line 30 > > > > > > Hit the same linter-using-old-virtualenv error when applying this patch > > as described in: >

Re: Review Request 51052: Make mesos-docker-execute understand cgroups_enable_cfs.

> On Oct. 18, 2016, 4:49 a.m., Jie Yu wrote: > > src/docker/docker.hpp, line 51 > > > > > > Having this as a global knob sounds really weird to me. I know you want > > to work around the Docker::run GMock issue. But

Re: Review Request 52939: Updated CLI pylint configuration to allow 2 character variable names.

> On Oct. 17, 2016, 6 p.m., Joseph Wu wrote: > > src/cli_new/pylint.config, line 13 > > > > > > I see a single 2 character variable name `io` here: > > https://reviews.apache.org/r/52953/diff/1#0 -- Line 306 > >

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

> On Oct. 18, 2016, 3:14 a.m., James Peach wrote: > > 3rdparty/libprocess/src/tests/io_tests.cpp, line 284 > > > > > > Can you just make ``length`` type ``ssize_t``? `length` is `ssize_t` (set on line 235) - Aaron

Re: Review Request 52940: Updated CLI pylint configuration to allow up to 30 local variables.

> On Oct. 17, 2016, 6 p.m., Joseph Wu wrote: > > src/cli_new/pylint.config, line 14 > > > > > > Looks like `__enter_namespaces` in https://reviews.apache.org/r/52953/ > > is over the default limit of 15 by one variab

Re: Review Request 52941: Updated CLI pylint configuration to allow 0 public methods.

> On Oct. 17, 2016, 6 p.m., Joseph Wu wrote: > > src/cli_new/pylint.config, line 15 > > > > > > Is this used anywhere? I did not encounter a linting error for this in > > your chain. > > Kevin Klues wrote: > I

Review Request 52991: Updated CHANGELOG in preparation of 1.2.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52991/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos Descripti

Review Request 52992: Updated configure.ac in preparation of 1.2.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52992/ --- Review request for mesos and Alexander Rukletsov. Repository: mesos Descripti

Re: Review Request 52953: Added container 'exec' command to the CLI.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52953/ --- (Updated Oct. 18, 2016, 7:01 p.m.) Review request for mesos and Joseph Wu. Ch

Re: Review Request 52988: Fixed a typo in a comment.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52988/#review153153 --- Patch looks great! Reviews applied: [52988] Passed command: expo

Re: Review Request 52980: Updated os::mkdir() to take the mode as an input.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52980/#review153155 --- 3rdparty/stout/include/stout/os/mkdir.hpp (line 43)

Re: Review Request 52980: Updated os::mkdir() to take the mode as an input.

> On Oct. 18, 2016, 7:58 p.m., Alex Clemmer wrote: > > 3rdparty/stout/include/stout/os/mkdir.hpp, line 43 > > > > > > Permissions is a bit of a subtle issue in Mesos, which we've largely > > punted on. I'm a bit worr

Re: Review Request 52976: Split mesos test helpers into 'internal' and 'v1' namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52976/ --- (Updated Oct. 18, 2016, 8:04 p.m.) Review request for mesos, Anand Mazumdar and

Re: Review Request 52976: Split mesos test helpers into 'internal' and 'v1' namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52976/ --- (Updated Oct. 18, 2016, 8:13 p.m.) Review request for mesos, Anand Mazumdar and

Re: Review Request 52780: Added input and output functions for v1::CapabilityInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52780/#review153162 --- Ship it! Made some adjustment while commit. - Jie Yu On Oct.

Re: Review Request 52906: Simplify the comparison logic for `ExecutorInfo`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52906/#review153165 --- Ship it! Ship It! - Vinod Kone On Oct. 15, 2016, 5:06 a.m.,

Re: Review Request 52969: Removed redundant test.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52969/#review153166 --- Ship it! Ship It! - Vinod Kone On Oct. 18, 2016, 1:13 a.m.,

Re: Review Request 52980: Updated os::mkdir() to take the mode as an input.

> On Oct. 18, 2016, 7:58 p.m., Alex Clemmer wrote: > > 3rdparty/stout/include/stout/os/mkdir.hpp, line 43 > > > > > > Permissions is a bit of a subtle issue in Mesos, which we've largely > > punted on. I'm a bit worr

Re: Review Request 52693: Changed master to send TASK_UNKNOWN during reconciliation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52693/#review153168 --- Ship it! Ship It! - Vinod Kone On Oct. 18, 2016, 1:14 a.m.,

Re: Review Request 52980: Updated os::mkdir() to take the mode as an input.

> On Oct. 18, 2016, 7:58 p.m., Alex Clemmer wrote: > > 3rdparty/stout/include/stout/os/mkdir.hpp, line 43 > > > > > > Permissions is a bit of a subtle issue in Mesos, which we've largely > > punted on. I'm a bit worr

Re: Review Request 52719: Renamed a function for clarity.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52719/#review153171 --- Ship it! s/reconcileKnownTasks/reconcileKnownSlave/ in the desc

Re: Review Request 52991: Updated CHANGELOG in preparation of 1.2.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52991/#review153174 --- Ship it! Ship It! - Alexander Rukletsov On Oct. 18, 2016, 6:

Re: Review Request 52719: Renamed a function for clarity.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52719/ --- (Updated Oct. 18, 2016, 9:07 p.m.) Review request for mesos and Vinod Kone. C

Re: Review Request 52659: Changed master to send TASK_DROPPED for task launch errors.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52659/ --- (Updated Oct. 18, 2016, 9:09 p.m.) Review request for mesos and Vinod Kone. C

Re: Review Request 52992: Updated configure.ac in preparation of 1.2.0.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52992/#review153177 --- Ship it! Ship It! - Alexander Rukletsov On Oct. 18, 2016, 6:

Review Request 52997: Improve Socket::connect error message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52997/ --- Review request for mesos and Michael Park. Bugs: MESOS-6412 https://issues.

Re: Review Request 52997: Improve Socket::connect error message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52997/ --- (Updated Oct. 18, 2016, 9:37 p.m.) Review request for mesos and Michael Park.

Re: Review Request 52997: Improve Socket::connect error message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52997/#review153179 --- 3rdparty/libprocess/src/poll_socket.cpp (line 114)

Re: Review Request 52997: Improve Socket::connect error message.

> On Oct. 18, 2016, 9:41 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/src/poll_socket.cpp, line 114 > > > > > > Missing reference. I wanted a copy (at least the lambda needs to take a copy), since ``con

Re: Review Request 50380: Added new benchmark test for port resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50380/#review153183 --- Patch looks great! Reviews applied: [52975, 50551, 52769, 51033,

Re: Review Request 52997: Improve Socket::connect error message.

> On Oct. 18, 2016, 9:41 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/src/poll_socket.cpp, line 114 > > > > > > Missing reference. > > James Peach wrote: > I wanted a copy (at least the lambda needs

Re: Review Request 52997: Improve Socket::connect error message.

On Oct. 18, 2016, 9:41 p.m., James Peach wrote: > > Can you also follow up with a review to change the logging level in > > libprocess for failing to close the socket if we fail to connect? > > Maybe it should be `VLOG(1)`? Thoughts? > > James Peach wrote: > I actually removed the `VLOG` th

Re: Review Request 52976: Split mesos test helpers into 'internal' and 'v1' namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52976/#review153167 --- src/tests/api_tests.cpp (line 838)

Re: Review Request 52600: Enable multiple field based authorization in the authorizer interface.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52600/#review153194 --- Seems you got a lot more within this review than planned :) - Til

Re: Review Request 52997: Improve Socket::connect error message.

> On Oct. 18, 2016, 9:41 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/src/poll_socket.cpp, line 114 > > > > > > Missing reference. > > James Peach wrote: > I wanted a copy (at least the lambda needs

Re: Review Request 52997: Improve Socket::connect error message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52997/ --- (Updated Oct. 18, 2016, 10:43 p.m.) Review request for mesos and Michael Park.

Review Request 53005: Updated release guide.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53005/ --- Review request for mesos and Till Toenshoff. Repository: mesos Description --

Re: Review Request 52720: Clarified a comment.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52720/#review153204 --- Fix it, then Ship it! src/master/master.cpp (line 6793)

Re: Review Request 52721: Fixed typo in log message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52721/#review153205 --- Ship it! Ship It! - Vinod Kone On Oct. 11, 2016, 1:03 a.m.,

Re: Review Request 52722: Changed master to add `FrameworkInfo` to agent reconcilation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52722/#review153207 --- Ship it! Ship It! - Vinod Kone On Oct. 11, 2016, 1:04 a.m.,

Re: Review Request 52723: Changed agent to send TASK_DROPPED during reconciliation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52723/#review153209 --- Fix it, then Ship it! src/tests/master_slave_reconciliation_te

Re: Review Request 52740: Refactored some code into a separate function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52740/#review153210 --- Ship it! Ship It! - Vinod Kone On Oct. 12, 2016, 7:34 p.m.,

Re: Review Request 52746: Changed agent to send TASK_DROPPED for task launch failures.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52746/#review153211 --- Fix it, then Ship it! Wonder if we could use a helper that retu

Re: Review Request 52682: Cleaned up a few style issues in the capabilities isolator tests.

> On Oct. 17, 2016, 10:02 p.m., Jie Yu wrote: > > src/tests/containerizer/linux_capabilities_isolator_tests.cpp, line 100 > > > > > > const UsaImage& > > Benjamin Bannier wrote: > I believe we always pass fundam

Re: Review Request 52910: Added a constants.hpp for provisioner.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52910/#review153216 --- Ship it! Ship It! - Jie Yu On Oct. 15, 2016, 1:39 p.m., Qian

Re: Review Request 52976: Split mesos test helpers into 'internal' and 'v1' namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52976/#review153218 --- Patch looks great! Reviews applied: [52976] Passed command: expo

Review Request 53008: Added a nested container test for 3 levels of nesting.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53008/ --- Review request for mesos, Benjamin Hindman, Gilbert Song, and Kevin Klues. Repo

Re: Review Request 53008: Added a nested container test for 3 levels of nesting.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53008/#review153222 --- Ship it! Ship It! - Gilbert Song On Oct. 18, 2016, 5:13 p.m.

Re: Review Request 52720: Clarified a comment.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52720/ --- (Updated Oct. 19, 2016, 12:37 a.m.) Review request for mesos and Vinod Kone.

Re: Review Request 52723: Changed agent to send TASK_DROPPED during reconciliation.

> On Oct. 18, 2016, 11:15 p.m., Vinod Kone wrote: > > src/tests/slave_recovery_tests.cpp, line 3067 > > > > > > Curious why you didn't write a new test for partition aware framework > > like you did above? I was t

  1   2   >