Re: Review Request 44934: Updated mesos-execute to add support for Appc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44934/#review124370 --- src/cli/execute.cpp (line 317)

Re: Review Request 44672: Added normalize method to registry puller.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44672/#review124379 --- src/slave/containerizer/mesos/provisioner/docker/registry_puller.

Re: Review Request 45067: Updated the long-lived-framework example.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45067/#review124377 --- Fix it, then Ship it! Ship It! src/examples/long_lived_framew

Re: Review Request 42684: Multiple Disk: Added persistent volumes tests for `MOUNT` type.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42684/#review124373 --- Patch looks great! Reviews applied: [42470, 42471, 42472, 42473,

Re: Review Request 44975: Updated cgroups test cases for cgroups device support.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44975/#review124375 --- I ran the review by Ben again and sat with him to rework it to his

Re: Review Request 44135: Libprocess: Use Windows-compatible memory fence in logging.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44135/ --- (Updated March 18, 2016, 6:48 p.m.) Review request for mesos, Alex Naparu, Dani

Re: Review Request 44661: Deprecated the `docker_stop_timeout` flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44661/ --- (Updated March 18, 2016, 5:23 p.m.) Review request for mesos, Ben Mahler and Ti

Re: Review Request 43630: Especially updated scheduler tests to use the updated MesosTest helpers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43630/#review123864 --- Ship it! Ship It! - Michael Park On March 14, 2016, 9:32 p.m

Re: Review Request 44767: Added authentication information to master endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44767/ --- (Updated March 18, 2016, 4:48 p.m.) Review request for mesos, Adam B and Greg M

Re: Review Request 44514: Implemented prepare() method of "network/cni" isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44514/ --- (Updated March 17, 2016, 9:17 p.m.) Review request for mesos, Avinash sridharan

Re: Review Request 44758: Upgrade to clang-format-3.8 (MESOS-4906).

> On March 15, 2016, 9:24 p.m., Michael Park wrote: > > support/clang-format, lines 101-107 > > > > > > We can help out a little more here: > > > > The following code orders the headers: > > (1) C standa

Re: Review Request 44854: Added validation for executor's shutdown grace period.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44854/#review124290 --- Ship it! The change looks good but please add a test for this.

Re: Review Request 44553: Added authentication to agent HTTP endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44553/#review124161 --- Fix it, then Ship it! Minor suggestions src/tests/slave_tests

Re: Review Request 44907: Remove `Fetcher` in `DockerContainerizer::create`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44907/ --- (Updated March 16, 2016, 5:57 p.m.) Review request for mesos and Till Toenshoff

Review Request 44954: Regenerated master endpoint documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44954/ --- Review request for mesos, Adam B and Greg Mann. Bugs: MESOS-4934 https://is

Review Request 44932: Removed unused overload of `model` for `TaskInfo`.

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

Re: Review Request 44421: Added support for "overlay" keyword.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44421/#review124140 --- src/linux/fs.cpp (line 74)

Re: Review Request 44081: Stout: Moved `os::libraries::` namespace back to `stout/os.hpp`.

> On March 18, 2016, 7:27 p.m., Michael Park wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp, line 131 > > > > > > `setenv` in `stout/windows/os.hpp` is commented out and unimplemeneted. > > How d

Re: Review Request 42684: Multiple Disk: Added persistent volumes tests for `MOUNT` type.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42684/#review124120 --- Fix it, then Ship it! src/tests/persistent_volume_tests.cpp (l

Re: Review Request 44678: Modified basic HTTP authenticator creator to accept realm.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44678/ --- (Updated March 18, 2016, 12:20 a.m.) Review request for mesos, Adam B, Alexande

Re: Review Request 44288: Changed MasterDetector/Contender namespace.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44288/ --- (Updated March 18, 2016, 12:29 a.m.) Review request for mesos and Benjamin Hind

Review Request 45034: Removed superfluous blank line in slave/http.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45034/ --- Review request for mesos, Adam B and Greg Mann. Repository: mesos Description

Re: Review Request 44909: Update containerizer construction in docker_containerizer_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44909/#review123862 --- Patch looks great! Reviews applied: [44798, 44822, 44823, 44824,

Review Request 44907: Remove `Fetcher` in `DockerContainerizer::create`.

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

Re: Review Request 44656: Introduced `KillPolicy` protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44656/ --- (Updated March 18, 2016, 5:14 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 44758: Upgrade to clang-format-3.8 (MESOS-4906).

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

Review Request 44899: Update containerizer construction in slave_recovery_tests.cpp.

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

Re: Review Request 44989: Fixed a race in the resource offers tests.

> On March 18, 2016, 9:47 a.m., Adam B wrote: > > "The test is slowed considerably" - Can you provide some stats on how fast > > it is before/after the patch? Good call - timing information has been added, and it actually wasn't as bad as I thought. - Greg -

Review Request 44903: Update containerizer construction in port_mapping_tests.cpp.

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

Re: Review Request 44997: Fixed how we detect C++11 compiler support.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44997/#review124132 --- Ship it! Ship It! - Joris Van Remoortere On March 17, 2016,

Re: Review Request 44766: Enabled Authentication information in endpoint HELP.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44766/ --- (Updated March 17, 2016, 5:01 p.m.) Review request for mesos, Adam B and Greg M

Re: Review Request 44974: Added device support in cgroups abstraction.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44974/#review124374 --- I ran the review by Ben again and sat with him to rework it to his

Re: Review Request 44139: Windows:[2/2] Lifted socket API into Stout.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44139/ --- (Updated March 17, 2016, 6:52 p.m.) Review request for mesos, Alex Naparu, Dani

Review Request 45015: Windows: Fixed bug causing `os::exists` to report invalid paths exist.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45015/ --- Review request for mesos, Alex Naparu, Daniel Pravat, Artem Harutyunyan, Joris V

Re: Review Request 44994: Added a test for executor shutdown grace period.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44994/ --- (Updated March 18, 2016, 5:13 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 44553: Added authentication to agent HTTP endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44553/ --- (Updated March 17, 2016, 10:56 p.m.) Review request for mesos, Adam B, Alexande

Re: Review Request 44675: Updated `/metrics/snapshot` endpoint to use `jsonify`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44675/#review124039 --- Ship it! Ship It! - Michael Park On March 10, 2016, 9:59 p.m

Re: Review Request 44954: Regenerated master endpoint documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44954/ --- (Updated March 18, 2016, 4:48 p.m.) Review request for mesos, Adam B and Greg M

Re: Review Request 44656: Introduced `KillPolicy` protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44656/ --- (Updated March 17, 2016, 11:36 p.m.) Review request for mesos and Ben Mahler.

Review Request 44896: Update containerizer construction in disk_quota_tests.cpp.

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

Re: Review Request 44985: Fix the broken ProvisionerDockerPullerTest on Centos7.

> On March 17, 2016, 10:12 p.m., Gilbert Song wrote: > > src/tests/containerizer/provisioner_docker_tests.cpp, line 401 > > > > > > `_NonShellCommand`? I'll rename it to SimpleCommand. - Jie

Re: Review Request 44655: Made `shutdown_grace_period` configurable in `ExecutorInfo`.

> On March 15, 2016, 8:11 p.m., Ben Mahler wrote: > > include/mesos/v1/executor/executor.proto, lines 53-55 > > > > > > Now that the period within the Shutdown message was removed, we should > > probably have a TODO

Re: Review Request 44985: Fix the broken ProvisionerDockerPullerTest on Centos7.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44985/ --- (Updated March 17, 2016, 10 p.m.) Review request for mesos, Bernd Mathiske and

Re: Review Request 37168: Add an example framework using dynamic reservation.

> On March 16, 2016, 6:27 p.m., Greg Mann wrote: > > src/examples/dynamic_reservation_framework.cpp, line 119 > > > > > > Could you clarify for me how the slave gets into the RESERVED state the > > first time? It s

Re: Review Request 44851: Renamed an allocator metric.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44851/#review124053 --- src/master/allocator/mesos/metrics.hpp (lines 40 - 41)

Re: Review Request 44706: Implemented isolate() method of "network/cni" isolator.

> On March 12, 2016, 2:45 a.m., Avinash sridharan wrote: > > src/slave/containerizer/mesos/isolators/network/cni.cpp, line 358 > > > > > > If we block here, it will lock the isolator, and prevent the > > `MesosConta

Re: Review Request 44553: Added authentication to agent HTTP endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44553/ --- (Updated March 18, 2016, 7:12 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 44657: Used `KillPolicy` and shutdown grace period in command executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44657/#review124359 --- src/launcher/executor.cpp (lines 496 - 499)

Re: Review Request 45040: Added a test for task's kill policy.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45040/#review124343 --- Thanks for the test! Although I wouldn't want to ship this unless

Re: Review Request 44755: Added getAppcImage for Appc provisioning tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44755/#review124001 --- src/tests/containerizer/provisioner_appc_tests.cpp (lines 67 - 73

Re: Review Request 43615: Update test suite to use the reworked MesosTest helpers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43615/#review123894 --- src/tests/containerizer/port_mapping_tests.cpp (line 2086)

Re: Review Request 44989: Fixed a race in the resource offers tests.

> On March 18, 2016, 12:16 a.m., Neil Conway wrote: > > src/tests/resource_offers_tests.cpp, line 63 > > > > > > Style-wise, do we want all tests to resume if they initially pause it? > > I think we do a mix of both.

Re: Review Request 44810: Documentation change for the update quota feature.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44810/#review123941 --- docs/quota.md (lines 157 - 163)

Re: Review Request 37168: Add an example framework using dynamic reservation.

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

Re: Review Request 44934: Updated mesos-execute to add support for Appc.

> On March 17, 2016, 2:16 a.m., Guangya Liu wrote: > > src/cli/execute.cpp, lines 251-281 > > > > > > What is the behavior of this if with mesos containerizer but without a > > docker/appc image? > > > > Th

Re: Review Request 44758: Upgrade to clang-format-3.8 (MESOS-4906).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44758/#review124038 --- Ship it! The patch looks good to me. Would you like to send an

Re: Review Request 33174: Fix for docker not configuring CFS quotas correctly

> On March 16, 2016, 4:27 p.m., Jie Yu wrote: > > Ship It! > > Jie Yu wrote: > Can you confirm that this patch works in your environment? Actually, it looks like the change to use update() instead of _update introduced a bug (the cgroups no longer get updated because the resources don't c

Re: Review Request 44909: Update containerizer construction in docker_containerizer_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44909/#review123926 --- Patch looks great! Reviews applied: [44798, 44822, 44823, 44824,

Re: Review Request 43629: Especially updated tests to use the updated MesosTest helpers.

> On March 16, 2016, 5:48 a.m., Michael Park wrote: > > src/tests/fetcher_cache_tests.cpp, lines 167-169 > > > > > > Could you explain this NOTE? It looks like we do `fetcherProcess = new > > MockFetcherProcess();`

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

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

Re: Review Request 44902: Update containerizer construction in memory_pressure_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44902/ --- (Updated March 16, 2016, 5:57 p.m.) Review request for mesos and Till Toenshoff

Re: Review Request 44435: Fixed a bug that causes the task stuck in staging state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44435/ --- (Updated March 19, 2016, 3:35 a.m.) Review request for mesos and Jie Yu. Bugs

Re: Review Request 44435: Fixed a bug that causes the task stuck in staging state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44435/#review124251 --- src/slave/containerizer/mesos/containerizer.cpp (line 674)

Re: Review Request 45033: Add a commit hook for checking non-ascii characters (MESOS-4033).

> On March 18, 2016, 7:03 p.m., haosdent huang wrote: > > docs/versioning.md, line 85 > > > > > > Any reason we need change here? > > Yong Tang wrote: > Hi haosdent, the left is one unicode (U+2026) of "Horizonta

Re: Review Request 44547: Added functions in promises to the collect header.

> On March 18, 2016, 2:05 p.m., Joseph Wu wrote: > > 3rdparty/libprocess/include/process/collect.hpp, line 260 > > > > > > Missed a couple extranous spaces here (i.e. `>* >*`). > > Anurag Singh wrote: > Thanks.

Re: Review Request 44657: Used `KillPolicy` and shutdown grace period in command executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44657/#review124352 --- Fix it, then Ship it! src/launcher/executor.cpp (line 924)

Re: Review Request 44709: Allowed unknown flags in command and docker executors.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44709/#review124330 --- src/docker/executor.cpp (lines 588 - 590)

Re: Review Request 44657: Used `KillPolicy` and shutdown grace period in command executor.

> On March 15, 2016, 10:25 p.m., Ben Mahler wrote: > > src/launcher/executor.cpp, line 499 > > > > > > I'd expect that the buffer is in addition to dealing with the reap > > interval, since the reap interval is not

Review Request 45057: Made `unzip` overwrite existing files without prompting.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45057/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4885 https://issues.apache

Re: Review Request 44993: Updated `TestContainerizer` to support default actions.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44993/#review124305 --- I don't have enough context on why you had to make this change, ca

Re: Review Request 44656: Introduced `KillPolicy` protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44656/#review124339 --- Fix it, then Ship it! include/mesos/mesos.proto (lines 354 - 3

Re: Review Request 40988: stout: Fixed comments for numify(), cleaned up some test code.

> On March 18, 2016, 11:13 p.m., Cong Wang wrote: > > Can you separate the non-numify() change from the numify() one? It is best > > to do one thing per commit. > > Neil Conway wrote: > This change was committed a few months ago. > > Cong Wang wrote: > I know it, I noticed this one bec

Re: Review Request 44762: [WIP][PROPOSAL_1]Add CgroupsIsolator. This is only used for discussion.

> On March 19, 2016, 12:39 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp, line 363 > > > > > > I think what you can do is to save a future in each Subsystem. > > > > I a

Re: Review Request 44836: Update TaskId in long_lived_framework to use fixed length taskid.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44836/#review124364 --- src/examples/long_lived_framework.cpp (line 88)

Re: Review Request 40988: stout: Fixed comments for numify(), cleaned up some test code.

> On March 18, 2016, 11:13 p.m., Cong Wang wrote: > > Can you separate the non-numify() change from the numify() one? It is best > > to do one thing per commit. > > Neil Conway wrote: > This change was committed a few months ago. I know it, I noticed this one because of 45011. And my comme

Re: Review Request 45000: MESOS-3902: Fix in location header during redirect from non-leader.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45000/#review124316 --- src/master/http.cpp (line 958)

Re: Review Request 44707: Added validation for task's kill policy.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44707/#review124341 --- Ship it! Please pull out the validation test into a simple unit

Re: Review Request 44762: [WIP][PROPOSAL_1]Add CgroupsIsolator. This is only used for discussion.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44762/#review124340 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Re: Review Request 44655: Made `shutdown_grace_period` configurable in `ExecutorInfo`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44655/#review124338 --- include/mesos/mesos.proto (line 445)

Re: Review Request 44435: Fixed a bug that causes the task stuck in staging state.

> On March 18, 2016, 7 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/containerizer.cpp, line 674 > > > > > > Thanks for catching the bug. I think this is a pretty bad bug which > > might worth cutting 0.28.1

Re: Review Request 44669: Added createFromModule methods to MasterContender and MasterDetector.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44669/#review124282 --- src/master/detectors/detector.cpp (line 126)

Re: Review Request 44994: Added a test for executor shutdown grace period.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44994/#review124308 --- Thanks for the test! Main high level comments are to pull out the

Re: Review Request 44287: Added MasterContender and MasterDetector abstract classes.

> On March 18, 2016, 9:01 p.m., Joseph Wu wrote: > > Sorry for the delay... > > > > As Vinod suggested, you might want to consider reaching out to another > > shepherd (as far as I can tell, BenH is not likely to have time in the > > foreseeable future). (And in case you don't know, I'm not a

Re: Review Request 44854: Added validation for executor's shutdown grace period.

> On March 18, 2016, 9:33 p.m., Ben Mahler wrote: > > The change looks good but please add a test for this. > > > > Ideally there could be a stateless validateExecutorInfo, in addition to the > > one that needs the master state. In the absence of this, you can write a > > test that spins up a

Re: Review Request 40988: stout: Fixed comments for numify(), cleaned up some test code.

> On March 18, 2016, 11:13 p.m., Cong Wang wrote: > > Can you separate the non-numify() change from the numify() one? It is best > > to do one thing per commit. This change was committed a few months ago. - Neil --- This is an automati

Re: Review Request 45011: Fix numify() to handle negative numbers consistently (MESOS-4070).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45011/#review124314 --- Ship it! Ship It! - Neil Conway On March 18, 2016, 10:49 p.m

Re: Review Request 45067: Updated the long-lived-framework example.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45067/#review124351 --- src/examples/long_lived_framework.cpp (line 102)

Review Request 45067: Updated the long-lived-framework example.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45067/ --- Review request for mesos, Anand Mazumdar, Artem Harutyunyan, and Kevin Klues. R

Re: Review Request 44945: Add autoconf tests for XFS project quotas.

> On March 18, 2016, 11:36 a.m., Jiang Yan Xu wrote: > > configure.ac, line 923 > > > > > > Sorry this should have been a continued discussion on > > https://reviews.apache.org/r/44342/#comment184051 but anyways: >

Re: Review Request 44421: Added support for "overlay" keyword.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44421/ --- (Updated 三月 18, 2016, 4:22 p.m.) Review request for mesos, haosdent huang, Jie

Re: Review Request 44660: Used `KillPolicy` and shutdown grace period in docker executor.

> On March 15, 2016, 11:21 p.m., Ben Mahler wrote: > > src/docker/executor.cpp, lines 657-663 > > > > > > Hm.. ideally we could ignore the docker stop flag if a kill policy was > > set, because the user is being exp

Re: Review Request 40988: stout: Fixed comments for numify(), cleaned up some test code.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40988/#review124320 --- Can you separate the non-numify() change from the numify() one? It

Review Request 45004: Fixed a memory leak in the scheduler driver.

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

Re: Review Request 37168: Add an example framework using dynamic reservation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/ --- (Updated March 17, 2016, 2:15 p.m.) Review request for mesos, Greg Mann, Joerg

Re: Review Request 44622: Introduced a protobuf message "NetworkResult".

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44622/#review123977 --- Fix it, then Ship it! Ship It! src/slave/containerizer/mesos/

Re: Review Request 43615: Update test suite to use the reworked MesosTest helpers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43615/#review123867 --- Ship it! On my machine, full test-suite remains to take about 1

Re: Review Request 44553: Added authentication to agent HTTP endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44553/ --- (Updated March 18, 2016, 7:16 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 45022: Windows: Add Windows-friendly implementation of `rm.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45022/ --- (Updated March 18, 2016, 7:39 a.m.) Review request for mesos, Alex Naparu, Dani

Re: Review Request 44553: Added authentication to agent HTTP endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44553/#review124194 --- src/slave/http.cpp (line 54)

Re: Review Request 44944: Handled chunked responses in docker URI fetcher.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44944/#review123979 --- Ship it! Ship It! - Timothy Chen On March 17, 2016, 1:50 a.m

  1   2   >