Re: Review Request 38750: Updated changelog for 0.25.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38750/#review100544 --- Patch looks great! Reviews applied: [38750] All tests passed. -

Re: Review Request 38747: Adding digest utilities

> On Sept. 25, 2015, 6:24 a.m., Alex Clemmer wrote: > > Hey, it doesn't look like `src/tests/digest_tests.cpp` is being added to > > the `CMakeLists.txt` file in `3rdparty/libprocess/src/tests`. Is that > > correct? If not, could we please add it there as well? > > > > (I have forgotten to add

Re: Review Request 37999: Implemented http::AuthenticatorManager

> On Sept. 23, 2015, 5:39 p.m., Alex Clemmer wrote: > > Can we please add this file to the CMakeLists.txt file in > > 3rdparty/libprocess/src as well? > > Alexander Rojas wrote: > The included file is a header and so far cmake file doesn't seem to care > about the include directory (can yo

Re: Review Request 38747: Adding digest utilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38747/#review100539 --- Hey, it doesn't look like `src/tests/digest_tests.cpp` is being add

Review Request 38756: CMake: Add support for compiling the agent with CMake.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38756/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Review Request 38754: CMake: Add build/configure/install logic for Zookeeper.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38754/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Review Request 38755: CMake: Move compiler configuration logic to its own file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38755/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Review Request 38753: CMake: Transition Stout tests to use new third-party build scripts.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38753/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Review Request 38752: CMake: Transition Mesos to use new third-party build scripts.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38752/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Review Request 38751: CMake: Pull third-party configuration logic into its own .cmake file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38751/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Re: Review Request 38747: Adding digest utilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38747/ --- (Updated Sept. 25, 2015, 5:46 a.m.) Review request for mesos, Gilbert Song and

Re: Review Request 38747: Adding digest utilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38747/ --- (Updated Sept. 25, 2015, 5:43 a.m.) Review request for mesos, Gilbert Song and

Re: Review Request 38747: Adding digest utilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38747/#review100530 --- Bad patch! Reviews applied: [38443, 38579, 38580, 38747] Failed c

Review Request 38750: Updated changelog for 0.25.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38750/ --- Review request for mesos and Joris Van Remoortere. Repository: mesos Descript

Re: Review Request 38749: Added `image_providers` flags to configuration.md.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38749/#review100528 --- Ship it! Ship It! - Michael Park On Sept. 25, 2015, 4:57 a.m.,

Re: Review Request 38749: Added `image_providers` flags to configuration.md.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38749/#review100527 --- Ship it! Ship It! - Niklas Nielsen On Sept. 24, 2015, 9:57 p.m.

Re: Review Request 38746: Added TaskStatus::Reason to containerizer Termination message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38746/#review100526 --- Patch looks great! Reviews applied: [38746] All tests passed. -

Review Request 38749: Added `image_providers` flags to configuration.md.

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

Re: Review Request 38338: Enhanced option for Docker cli volume plugin.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38338/#review100520 --- Patch looks great! Reviews applied: [38338] All tests passed. -

Re: Review Request 38342: Add JSON::protobuf for google::protobuf::RepeatedPtrField (stout part)

> On Sept. 24, 2015, 3:52 p.m., Jan Schlicht wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp, line 609 > > > > > > s/protobuf(const T& message)/protobuf(const google::protobuf::Message& > >

Re: Review Request 38738: Make common public header type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38738/#review100518 --- Bad patch! Reviews applied: [38726] Failed command: ./support/app

Review Request 38747: Adding digest utilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38747/ --- Review request for mesos, Gilbert Song and Timothy Chen. Repository: mesos De

Re: Review Request 38654: Added digest verifier for image blobs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38654/#review100517 --- Patch looks great! Reviews applied: [38443, 38579, 38580, 38654]

Review Request 38746: Added TaskStatus::Reason to containerizer Termination message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38746/ --- Review request for mesos, Ben Mahler, Timothy Chen, and Vinod Kone. Bugs: MESOS

Re: Review Request 38646: Added WIP note for Executor endpoint in changelog

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38646/#review100515 --- Patch looks great! Reviews applied: [38646] All tests passed. -

Re: Review Request 38743: files: fix potential fd leaks under error conditions.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38743/#review100511 --- Ship it! Can't wait to have a ref-counted abstraction to avoid thi

Re: Review Request 38338: Enhanced option for Docker cli volume plugin.

> On Sept. 24, 2015, 11:28 p.m., Vaibhav Khanduja wrote: > > src/docker/docker.cpp, line 422 > > > > > > volumn - "volume" fixed. - haosdent --- This is an

Re: Review Request 38338: Enhanced option for Docker cli volume plugin.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38338/ --- (Updated Sept. 25, 2015, 1:12 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 37903: stout: Fix bug in IPNetwork::create() with zero prefix.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37903/#review100509 --- Bad review! Reviews applied: [] Error: No reviewers specified. P

Re: Review Request 38738: Make common public header type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38738/ --- (Updated Sept. 25, 2015, 12:45 a.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 38468: docs: Added discussion of finding a shepherd.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38468/#review100507 --- Patch looks great! Reviews applied: [38468] All tests passed. -

Re: Review Request 38654: Added digest verifier for image blobs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38654/ --- (Updated Sept. 25, 2015, 12:38 a.m.) Review request for mesos, Gilbert Song and

Re: Review Request 38705: Added support for applying a review chain (apply-reviews.py).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38705/#review100505 --- Mostly just python style and commenting. My python is a bit rusty

Re: Review Request 38646: Added WIP note for Executor endpoint in changelog

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38646/ --- (Updated Sept. 25, 2015, 12:09 a.m.) Review request for mesos and Vinod Kone.

Re: Review Request 38731: Make common resources symmetrical to v1 resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38731/ --- (Updated Sept. 25, 2015, 12:07 a.m.) Review request for mesos and Benjamin Hind

Re: Review Request 38733: Make common type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38733/ --- (Updated Sept. 25, 2015, 12:06 a.m.) Review request for mesos and Benjamin Hind

Re: Review Request 38738: Make common public header type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38738/#review100504 --- Ship it! Let's actually keep the comment (but change the wording a

Re: Review Request 38607: Pulled out an encode function for http::Request encoding.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38607/#review100500 --- Ship it! Ship It! - Vinod Kone On Sept. 22, 2015, 6:18 a.m., Be

Re: Review Request 38734: Cleaned up function signatures to use Option.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38734/#review100501 --- Patch looks great! Reviews applied: [38734] All tests passed. -

Re: Review Request 38645: Fixed Flaky Executor HTTP tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38645/ --- (Updated Sept. 24, 2015, 11:48 p.m.) Review request for mesos, Isabel Jimenez a

Re: Review Request 38645: Fixed Flaky Executor HTTP tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38645/#review100499 --- Ship it! Ship It! - Vinod Kone On Sept. 24, 2015, 11:24 p.m., A

Review Request 38743: files: fix potential fd leaks under error conditions.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38743/ --- Review request for mesos. Repository: mesos Description --- files: fix p

Re: Review Request 38726: Make common attributes symmetrical to v1 attributes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38726/#review100498 --- Ship it! Ship It! - Benjamin Hindman On Sept. 24, 2015, 6:30 p.

Re: Review Request 38731: Make common resources symmetrical to v1 resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38731/#review100496 --- Ship it! Ship It! - Benjamin Hindman On Sept. 24, 2015, 8:20 p.

Re: Review Request 38733: Make common type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38733/#review100495 --- Ship it! But let's keep the 'checkpoint' stuff in type_utils.cpp f

Re: Review Request 38727: Make common values symmetrical to v1 values.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38727/#review100497 --- Ship it! Ship It! - Benjamin Hindman On Sept. 24, 2015, 6:53 p.

Re: Review Request 38338: Enhanced option for Docker cli volume plugin.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38338/#review100494 --- src/docker/docker.cpp (line 422)

Re: Review Request 38645: Fixed Flaky Executor HTTP tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38645/ --- (Updated Sept. 24, 2015, 11:24 p.m.) Review request for mesos, Isabel Jimenez a

Re: Review Request 38646: Added WIP note for Executor endpoint in changelog

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38646/#review100493 --- Ship it! Ship It! - Vinod Kone On Sept. 22, 2015, 9:18 p.m., An

Re: Review Request 38645: Fixed Flaky Executor HTTP tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38645/ --- (Updated Sept. 24, 2015, 11:09 p.m.) Review request for mesos, Isabel Jimenez a

Re: Review Request 38137: Added Docker provisioner, store and local puller

> On Sept. 24, 2015, 9 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/provisioner/docker/local_puller.hpp, line 40 > > > > > > s/docker_discovery_local_dir/docker_local_archives_dir/ > > > > I suggest

Re: Review Request 38137: Added Docker provisioner, store and local puller

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review100492 --- Patch looks great! Reviews applied: [38137] All tests passed. -

Review Request 38738: Make common public header type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38738/ --- Review request for mesos, Benjamin Hindman and Ben Mahler. Bugs: MESOS-3510

Re: Review Request 38731: Make common resources symmetrical to v1 resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38731/#review100485 --- Ship it! Ship It! - Ben Mahler On Sept. 24, 2015, 8:20 p.m., Jo

Re: Review Request 38726: Make common attributes symmetrical to v1 attributes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38726/#review100480 --- Ship it! Ship It! - Ben Mahler On Sept. 24, 2015, 6:30 p.m., Jo

Re: Review Request 38729: Do not retry close on EINTR.

> On Sept. 24, 2015, 8:55 p.m., Jie Yu wrote: > > 3rdparty/libprocess/src/subprocess.cpp, line 133 > > > > > > Should we wrap it with #ifdef linux? > > Ben Mahler wrote: > Sadly, it is more involved than just an

Re: Review Request 38645: Fixed Flaky Executor HTTP tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38645/#review100477 --- src/tests/executor_http_api_tests.cpp (line 95)

Re: Review Request 38729: Do not retry close on EINTR.

> On Sept. 24, 2015, 8:55 p.m., Jie Yu wrote: > > 3rdparty/libprocess/src/subprocess.cpp, line 133 > > > > > > Should we wrap it with #ifdef linux? Sadly, it is more involved than just an ifdef :( For example, on O

Re: Review Request 38731: Make common resources symmetrical to v1 resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38731/#review100474 --- Patch looks great! Reviews applied: [38726, 38727, 38731] All tes

Re: Review Request 38729: Do not retry close on EINTR.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38729/#review100472 --- Ship it! Ship It! - Vinod Kone On Sept. 24, 2015, 7:41 p.m., Be

Re: Review Request 38730: Do not retry close on EINTR.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38730/#review100473 --- Ship it! Ship It! - Vinod Kone On Sept. 24, 2015, 7:41 p.m., Be

Re: Review Request 37903: stout: Fix bug in IPNetwork::create() with zero prefix.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37903/ --- (Updated Sept. 24, 2015, 9:29 p.m.) Review request for mesos. Changes ---

Re: Review Request 38468: docs: Added discussion of finding a shepherd.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38468/ --- (Updated Sept. 24, 2015, 9:29 p.m.) Review request for mesos, Adam B, Benjamin

Review Request 38734: Cleaned up function signatures to use Option.

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

Review Request 38733: Make common type_utils symmetrical to v1 mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38733/ --- Review request for mesos and Benjamin Hindman. Bugs: MESOS-3510 https://iss

Re: Review Request 38137: Added Docker provisioner, store and local puller

> On Sept. 23, 2015, 12:50 a.m., Jiang Yan Xu wrote: > > src/slave/containerizer/provisioner/docker/puller.hpp, line 60 > > > > > > If we use LinkedHashMap we don't need to create another struct right? > > Timothy C

Re: Review Request 38137: Added Docker provisioner, store and local puller

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review100449 --- Ship it! Thanks for being patient. Please commit this! src/slave

Re: Review Request 38729: Do not retry close on EINTR.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38729/#review100463 --- 3rdparty/libprocess/src/subprocess.cpp (line 133)

Re: Review Request 38730: Do not retry close on EINTR.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38730/#review100462 --- Patch looks great! Reviews applied: [38729, 38730] All tests pass

Review Request 38731: Make common resources symmetrical to v1 resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38731/ --- Review request for mesos and Benjamin Hindman. Bugs: MESOS-3510 https://iss

Re: Review Request 38137: Added Docker provisioner, store and local puller

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/ --- (Updated Sept. 24, 2015, 8:02 p.m.) Review request for mesos, Jie Yu, Jojy Varg

Re: Review Request 38727: Make common values symmetrical to v1 values.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38727/#review100457 --- Patch looks great! Reviews applied: [38726, 38727] All tests pass

Re: Review Request 38728: Cgroups Test Filters aborts test.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38728/#review100454 --- Ship it! Ship It! - Ben Mahler On Sept. 24, 2015, 7:35 p.m., Pa

Review Request 38729: Do not retry close on EINTR.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38729/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-3512 https://issues.ap

Review Request 38730: Do not retry close on EINTR.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38730/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-3512 https://issues.ap

Review Request 38728: Cgroups Test Filters aborts test.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38728/ --- Review request for mesos, Ben Mahler and Ian Downes. Bugs: MESOS-3513 https

Re: Review Request 38137: Added Docker provisioner, store and local puller

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review100452 --- Ship it! LGTM! Let's get this committed! src/slave/containerizer

Re: Review Request 38137: Added Docker provisioner, store and local puller

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review100451 --- Ship it! Ship It! - Jojy Varghese On Sept. 24, 2015, 9:21 a.m.,

Review Request 38727: Make common values symmetrical to v1 values.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38727/ --- Review request for mesos and Benjamin Hindman. Bugs: MESOS-3510 https://iss

Re: Review Request 38726: Make common attributes symmetrical to v1 attributes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38726/ --- (Updated Sept. 24, 2015, 6:30 p.m.) Review request for mesos and Benjamin Hindm

Review Request 38726: Make common attributes symmetrical to v1 attributes.

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

Review Request 38725: Fixed a typo for Agent help string for --launcher.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38725/ --- Review request for mesos, Jie Yu and Niklas Nielsen. Repository: mesos Descri

Re: Review Request 38697: Updated documentation strings for --launcher flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38697/#review100439 --- Ship it! Ship It! - Jie Yu On Sept. 24, 2015, 6:44 a.m., Kapil

Re: Review Request 37999: Implemented http::AuthenticatorManager

> On Sept. 23, 2015, 5:39 p.m., Alex Clemmer wrote: > > Can we please add this file to the CMakeLists.txt file in > > 3rdparty/libprocess/src as well? > > Alexander Rojas wrote: > The included file is a header and so far cmake file doesn't seem to care > about the include directory (can yo

Re: Review Request 37996: Added property manager

> On Sept. 23, 2015, 5:40 p.m., Alex Clemmer wrote: > > Can we please add these tests to the CMakeLists.txt file in > > 3rdparty/libprocess/3rdparty/stout/tests? > > Alexander Rojas wrote: > It was done already since the first version of this patch. Ah. I'm so sorry. I just kind of troll a

Re: Review Request 37993: Add explanatory comments for Allocator interface

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37993/#review100393 --- Looking almost perfect! Really appreciate that you diligently addre

Re: Review Request 38342: Add JSON::protobuf for google::protobuf::RepeatedPtrField (stout part)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38342/#review100413 --- 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp (lin

Re: Review Request 38335: Add JSON::protobuf for google::protobuf::RepeatedPtrField

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38335/ --- (Updated Sept. 24, 2015, 2:43 p.m.) Review request for mesos and Michael Park.

Re: Review Request 38342: Add JSON::protobuf for google::protobuf::RepeatedPtrField (stout part)

> On Sept. 15, 2015, 12:59 p.m., Jan Schlicht wrote: > > Please rebase your patch, latest commits in master lots of JSON related > > stuff. > > Jan Schlicht wrote: > Will review this RR after rebase. The code diff was uploaded after resolving the conflict - Klaus --

Re: Review Request 38342: Add JSON::protobuf for google::protobuf::RepeatedPtrField (stout part)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38342/ --- (Updated Sept. 24, 2015, 2:37 p.m.) Review request for mesos, Alexander Ruklets

Re: Review Request 38094: Added implementation of Http Basic authentication scheme.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38094/#review100398 --- Patch looks great! Reviews applied: [37714, 37996, 37997, 37998, 3

Re: Review Request 38051: Only update the task status when its old status is not terminal.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38051/#review100394 --- Patch looks great! Reviews applied: [38051] All tests passed. -

Re: Review Request 38094: Added implementation of Http Basic authentication scheme.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38094/ --- (Updated Sept. 24, 2015, 12:33 p.m.) Review request for mesos, Adam B, Benjamin

Re: Review Request 37999: Implemented http::AuthenticatorManager

> On Sept. 23, 2015, 7:39 p.m., Alex Clemmer wrote: > > Can we please add this file to the CMakeLists.txt file in > > 3rdparty/libprocess/src as well? The included file is a header and so far cmake file doesn't seem to care about the include directory (can you correct me if I'm wrong). Still,

Re: Review Request 38158: Refactored Value::Ranges coalesce().

> On Sept. 23, 2015, 6:29 p.m., Ben Mahler wrote: > > Where is the benchmark for this change? :( > > Joerg Schad wrote: > We benchmarked the code with a proprietary benchmark. I added > https://issues.apache.org/jira/browse/MESOS-3502 aiming at adding an > open-source benchmark for this.

Re: Review Request 38137: Added Docker provisioner, store and local puller

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review100387 --- Patch looks great! Reviews applied: [38137] All tests passed. -

Re: Review Request 37996: Added property manager

> On Sept. 23, 2015, 7:40 p.m., Alex Clemmer wrote: > > Can we please add these tests to the CMakeLists.txt file in > > 3rdparty/libprocess/3rdparty/stout/tests? It was done already since the first version of this patch. - Alexander --

Re: Review Request 37714: Updated Multimap and multihashmap so their signatures resemble that of hashmap and hashset.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37714/ --- (Updated Sept. 24, 2015, 12:05 p.m.) Review request for mesos, Joerg Schad, Mic

Re: Review Request 37997: Running PathInheritedProperties tests from libprocess Makefile

> On Sept. 23, 2015, 7:38 p.m., Alex Clemmer wrote: > > Can we add these tests to the CMakeLists.txt file in > > 3rdparty/libprocess/3rdparty/stout/tests/ as well? As I mentioned it was done in a previous patch. But the stout test with autotool are ran from the libprocess Makefile which forces

Re: Review Request 38051: Only update the task status when its old status is not terminal.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38051/ --- (Updated Sept. 24, 2015, 9:45 a.m.) Review request for mesos and Vinod Kone.

  1   2   >