Re: Review Request 36946: Factored out the pattern for URL generation in a fetcher test.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36946/#review93878 --- Ship it! Ship It! - Bernd Mathiske On July 31, 2015, 8:23 a.m.,

Re: Review Request 36773: Implemented a LRU entry selection criteria for cache eviction.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36773/ --- (Updated Aug. 3, 2015, 10:43 a.m.) Review request for mesos, Bernd Mathiske and

Re: Review Request 36847: Added HTTP Delete Method.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36847/#review93880 --- 3rdparty/libprocess/include/process/http.hpp (line 736)

Re: Review Request 36773: Implemented a LRU entry selection criteria for cache eviction.

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

Re: Review Request 36773: Implemented a LRU entry selection criteria for cache eviction.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36773/#review93886 --- Ship it! Ship It! - Joerg Schad On Aug. 3, 2015, 8:43 a.m., Jan

Re: Review Request 36048: Updated authorizer to allow for modularized implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36048/#review93884 --- Ship it! I think this looks ready for landing after fixing the rema

Re: Review Request 36773: Implemented a LRU entry selection criteria for cache eviction.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36773/#review93889 --- Ship it! Ship It! - Bernd Mathiske On Aug. 3, 2015, 1:43 a.m., J

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/#review93890 --- src/master/http.cpp (lines 517 - 521)

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/ --- (Updated Aug. 3, 2015, 11:55 a.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 36049: Added support for modularized Authorizer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36049/#review93898 --- Ship it! Ship It! - Till Toenshoff On Aug. 3, 2015, 9:47 a.m., A

Re: Review Request 36050: Added test authorizer module.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36050/#review93899 --- Ship it! src/tests/authorization_tests.cpp (line 25)

Re: Review Request 36048: Updated authorizer to allow for modularized implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36048/#review93900 --- include/mesos/authorizer/authorizer.hpp (line 34)

Re: Review Request 36049: Added support for modularized Authorizer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36049/#review93906 --- src/local/local.cpp (line 221)

Re: Review Request 36048: Updated authorizer to allow for modularized implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36048/#review93907 --- include/mesos/authorizer/authorizer.hpp (line 24)

Re: Review Request 36908: Added QuotaInfo Protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36908/#review93908 --- include/mesos/master/quota.proto (line 25)

Re: Review Request 36908: Added QuotaInfo Protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36908/#review93909 --- Ship it! Looks good to me. src/Makefile.am (lines 182 - 183)

Re: Review Request 36908: Added QuotaInfo Protobuf.

> On Aug. 3, 2015, 2:44 p.m., Bernd Mathiske wrote: > > include/mesos/master/quota.proto, line 38 > > > > > > limit, bound -> plural I would suggest to reword: "Add upper bounds limit of resources that ..." - Till

Re: Review Request 36911: Removed unnecessary using directive.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36911/ --- (Updated Aug. 3, 2015, 2:58 p.m.) Review request for mesos, Marco Massenzio and

Re: Review Request 36048: Updated authorizer to allow for modularized implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36048/#review93912 --- src/authorizer/authorizer.cpp (line 209)

Re: Review Request 36929: Fixed a few issues in test launcher header.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36929/#review93917 --- Ship it! Ship It! - Vinod Kone On July 31, 2015, 10:23 p.m., Jie

Re: Review Request 36956: Created a test abstraction for preparing test rootfs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36956/ --- (Updated Aug. 3, 2015, 5:18 p.m.) Review request for mesos, Ian Downes, Timothy

Re: Review Request 36911: Removed unnecessary using directive.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36911/#review93919 --- Ship it! Ship It! - Marco Massenzio On Aug. 3, 2015, 2:58 p.m.,

Re: Review Request 36404: Added support for peek() to process::io

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36404/ --- (Updated Aug. 3, 2015, 10:44 a.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 36847: Added HTTP Delete Method.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36847/#review93923 --- 3rdparty/libprocess/include/process/http.hpp (line 736)

Re: Review Request 36847: Added HTTP Delete Method.

> On Aug. 3, 2015, 5:47 p.m., Marco Massenzio wrote: > > 3rdparty/libprocess/include/process/http.hpp, line 736 > > > > > > please don't use abbreviations. > > `delete()` is the HTTP method name. > > > >

Re: Review Request 36847: Added HTTP Delete Method.

> On Aug. 3, 2015, 5:47 p.m., Marco Massenzio wrote: > > 3rdparty/libprocess/src/http.cpp, lines 927-929 > > > > > > I would add a statement in the (javadoc?) method's documentation, to > > the effect that a query o

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/ --- (Updated Aug. 3, 2015, 6:09 p.m.) Review request for mesos, Alexander Rukletsov

Re: Review Request 36819: Use setup.py in python cli package.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36819/#review93931 --- Ship it! This looks good to me. Provided that we test it on a coupl

Re: Review Request 36956: Created a test abstraction for preparing test rootfs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36956/#review93933 --- Patch looks great! Reviews applied: [36929, 36930, 36954, 36956] A

Re: Review Request 36404: Added support for peek() to process::io

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36404/ --- (Updated Aug. 3, 2015, 11:30 a.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 36987: Extend 'getFormValue' -> 'getFormValues' to parse input uniformly.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36987/#review93942 --- src/master/http.cpp (lines 361 - 362)

Re: Review Request 36847: Added HTTP Delete Method.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36847/ --- (Updated Aug. 3, 2015, 6:45 p.m.) Review request for mesos, Alexander Rukletsov

Re: Review Request 36987: Extend 'getFormValue' -> 'getFormValues' to parse input uniformly.

> On Aug. 3, 2015, 6:43 p.m., Ben Mahler wrote: > > src/master/http.cpp, lines 361-362 > > > > > > Do we even want to keep this? Note that the 'observe' has never been > > used. > > > > I've found this to b

Re: Review Request 36908: Added QuotaInfo Protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36908/ --- (Updated Aug. 3, 2015, 7:01 p.m.) Review request for mesos, Alexander Rukletsov

Review Request 37046: Merged registerFramework() and reregisterFramework().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37046/ --- Review request for mesos, Anand Mazumdar and Ben Mahler. Bugs: MESOS-3182 h

Re: Review Request 36048: Updated authorizer to allow for modularized implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36048/#review93951 --- I have a stupid question. Why shouldn't the authorizer.{hpp,proto} f

Re: Review Request 37024: Add an endpoint that exposes component version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37024/#review93947 --- Thanks! Let's include the git / build information as well. Another

Re: Review Request 37023: Add an endpoint that exposes component flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37023/#review93952 --- Ship it! Nice work! Just a couple of small fixes, notably we're try

Re: Review Request 36987: Extend 'getFormValue' -> 'getFormValues' to parse input uniformly.

> On Aug. 3, 2015, 6:43 p.m., Ben Mahler wrote: > > src/master/http.cpp, lines 361-362 > > > > > > Do we even want to keep this? Note that the 'observe' has never been > > used. > > > > I've found this to b

Re: Review Request 37046: Merged registerFramework() and reregisterFramework().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37046/#review93955 --- Ship it! src/master/master.cpp (lines 1719 - 1721)

Re: Review Request 36911: Removed unnecessary using directive.

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

Re: Review Request 36987: Extend 'getFormValue' -> 'getFormValues' to parse input uniformly.

> On Aug. 3, 2015, 6:43 p.m., Ben Mahler wrote: > > src/master/http.cpp, lines 361-362 > > > > > > Do we even want to keep this? Note that the 'observe' has never been > > used. > > > > I've found this to b

Re: Review Request 34427: AppC provisioner backend using bind mounts.

> On May 21, 2015, 12:29 p.m., Timothy Chen wrote: > > src/slave/containerizer/provisioners/appc/bind_backend.hpp, line 70 > > > > > > Should we make "rootfs" a constant somewhere? Yeah, I think there should be a paths

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/ --- (Updated Aug. 3, 2015, 8:19 p.m.) Review request for mesos, Anand Mazumdar, Ben

Re: Review Request 37007: Pass ExecutorID to task-status label decorator hook.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37007/#review93963 --- include/mesos/hook.hpp (lines 89 - 90)

Re: Review Request 36867: Add "labels" to FrameworkInfo.

> On July 27, 2015, 11:36 p.m., Adam B wrote: > > Great first patch. Thanks for updating FrameworkInfo on reregistration with > > the master too! > > A handful of nits in my first pass. I'll take another look once you've > > simplified the tests with Kapil's suggestions. Any updates here? :)

Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-2834 https://issues.ap

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/#review93968 --- Looks pretty good, thanks Paul! Just a couple bits of feedback aroun

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/#review93970 --- Is this ready to review? Mind updating the 'issues' accordingly? -

Re: Review Request 37046: Merged registerFramework() and reregisterFramework().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37046/ --- (Updated Aug. 3, 2015, 9:47 p.m.) Review request for mesos, Anand Mazumdar and

Re: Review Request 37046: Merged registerFramework() and reregisterFramework().

> On Aug. 3, 2015, 8:07 p.m., Ben Mahler wrote: > > src/master/master.cpp, lines 1719-1721 > > > > > > Hm.. for later, I suppose we'll want to have a metric helper for calls > > inside receive. We currently aren't

Review Request 37054: Moved ContainerInfo::Image definition to the top level.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37054/ --- Review request for mesos, Benjamin Hindman, Timothy Chen, and Vinod Kone. Repos

Review Request 37055: Added Image to Volume as one of the sources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37055/ --- Review request for mesos, Benjamin Hindman, Timothy Chen, and Vinod Kone. Repos

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

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

Re: Review Request 36867: Add "labels" to FrameworkInfo.

> On July 27, 2015, 11:36 p.m., Adam B wrote: > > Great first patch. Thanks for updating FrameworkInfo on reregistration with > > the master too! > > A handful of nits in my first pass. I'll take another look once you've > > simplified the tests with Kapil's suggestions. > > Niklas Nielsen wro

Re: Review Request 32700: Removed FrameworkID from FrameworkState.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32700/#review93988 --- Looks great! Just a couple of questions. Did you run `sudo make chec

Re: Review Request 36404: Added support for peek() to process::io

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36404/#review93994 --- Patch looks great! Reviews applied: [36999, 36646, 36404] All test

Re: Review Request 36814: Fill executor_id in state.json when task is run in CommandExecutor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36814/#review93995 --- Ship it! LGTM. I like the idea of not setting the executorId=taskId

Re: Review Request 37054: Moved ContainerInfo::Image definition to the top level.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37054/#review93997 --- Ship it! Ship It! - Timothy Chen On Aug. 3, 2015, 11:09 p.m., Ji

Re: Review Request 37054: Moved ContainerInfo::Image definition to the top level.

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

Re: Review Request 36847: Added HTTP Delete Method.

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

Review Request 37065: Fixed MemIsolatorTest failure on OSX.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37065/ --- Review request for mesos, Benjamin Hindman and switched to 'mcypark'. Bugs: MES

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37065/#review94001 --- Thanks for this Artem! I applied the patch and it works. Just a mino