Re: Review Request 39420: Added '--parent' option and made apply-review.sh call apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39420/ --- (Updated Nov. 9, 2015, 11:57 p.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 39420: Added '--parent' option and made apply-review.sh call apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39420/ --- (Updated Nov. 9, 2015, 11:56 p.m.) Review request for mesos, Joris Van Remoorte

Re: Review Request 39410: Added support for github to apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39410/ --- (Updated Nov. 9, 2015, 11:48 p.m.) Review request for mesos, Adam B, Joris Van

Re: Review Request 39410: Added support for github to apply-reviews.py.

> On Nov. 9, 2015, 8:11 p.m., Kapil Arya wrote: > > support/apply-reviews.py, lines 196-197 > > > > > > Can we enhance it as following: > > ``` > > amend=options['no_amend'] ? '' : '-e' > > cmd = 'git co

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38059/#review105801 --- Let's just try and make this chain a little easier to follow. Right

Re: Review Request 39636: FreeBSD: Add basic support to stout

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39636/#review105800 --- Bad patch! Reviews applied: [39634, 39635] Failed command: ./supp

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

> On Nov. 6, 2015, 2:58 p.m., Klaus Ma wrote: > > src/master/quota_handler.cpp, line 1 > > > > > > It seems other feature named file without "_handler"; any consideration > > for it? > > Joerg Schad wrote: > In

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/#review105798 --- Ship it! src/master/master.hpp (line 859)

Re: Review Request 40125: Fixed quoted strings in executor log messages on agent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40125/#review105797 --- Ship it! Ship It! - Joris Van Remoortere On Nov. 10, 2015, 5:46

Re: Review Request 39635: FreeBSD: Enable libprocess build and disable failing test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39635/ --- (Updated Nov. 10, 2015, 6:35 a.m.) Review request for mesos and Ian Downes. B

Re: Review Request 39636: FreeBSD: Add basic support to stout

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39636/ --- (Updated Nov. 10, 2015, 6:35 a.m.) Review request for mesos and Ian Downes. B

Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39634/ --- (Updated Nov. 10, 2015, 6:35 a.m.) Review request for mesos and Ian Downes. B

Re: Review Request 40062: MESOS-2315 Deprecate/Remove CommandInfo::ContainerInfo

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

Re: Review Request 40125: Fixed quoted strings in executor log messages on agent

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

Review Request 40125: Fixed quoted strings in executor log messages on agent

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

Re: Review Request 40062: MESOS-2315 Deprecate/Remove CommandInfo::ContainerInfo

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

Re: Review Request 40062: MESOS-2315 Deprecate/Remove CommandInfo::ContainerInfo

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40062/ --- (Updated Nov. 10, 2015, 5:16 a.m.) Review request for mesos and Adam B. Chang

Re: Review Request 40062: MESOS-2315 Deprecate/Remove CommandInfo::ContainerInfo

> On Nov. 9, 2015, 7:27 p.m., Joseph Wu wrote: > > include/mesos/mesos.proto, line 372 > > > > > > Should we also remove this from v1/mesos.proto? Fixed - Vaibhav

Re: Review Request 39043: Added support for HTTP Authentication in Mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39043/#review105788 --- Patch looks great! Reviews applied: [39276, 37998, 37999, 38000, 3

Re: Review Request 39043: Added support for HTTP Authentication in Mesos.

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

Re: Review Request 39410: Added support for github to apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39410/#review105786 --- support/apply-reviews.py (lines 187 - 188)

Re: Review Request 39043: Added support for HTTP Authentication in Mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39043/ --- (Updated Nov. 10, 2015, 5:03 a.m.) Review request for mesos, Adam B, Bernd Math

Re: Review Request 38950: Http Authenticators can be loaded as modules from mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38950/ --- (Updated Nov. 10, 2015, 5:02 a.m.) Review request for mesos, Adam B, Bernd Math

Re: Review Request 38627: Adds an overload of ModuleManager::create() allowing overriding parameters programatically

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38627/ --- (Updated Nov. 10, 2015, 5:01 a.m.) Review request for mesos, Adam B, Bernd Math

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 Nov. 10, 2015, 5 a.m.) Review request for mesos, Adam B, Benjamin Hind

Re: Review Request 38000: Added an API for libprocess users to interact with http::AuthenticatorManager

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38000/ --- (Updated Nov. 10, 2015, 4:59 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 37999: Implemented http::AuthenticatorManager

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37999/ --- (Updated Nov. 10, 2015, 4:59 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 37998: Made ProcessManager::handle() a void returning method.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37998/ --- (Updated Nov. 10, 2015, 4:58 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 39276: Fixed a bug in which under certains circumstances HTTP 1.1 Pipelining is not respected.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39276/ --- (Updated Nov. 10, 2015, 4:58 a.m.) Review request for mesos, Anand Mazumdar, Be

Re: Review Request 39250: Puller refactor: moved untar to a common place

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review105780 --- Bad patch! Reviews applied: [39832, 39882, 39839, 38579, 39015, 39

Re: Review Request 40120: Refactored Docker registry puller.

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

Re: Review Request 39250: Puller refactor: moved untar to a common place

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

Review Request 40120: Refactored Docker registry puller.

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

Re: Review Request 39331: Support docker local store pull image simultaneously

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39331/#review105773 --- src/slave/containerizer/mesos/provisioner/docker/store.cpp (line 1

Re: Review Request 40044: Remove unused parameter in moveLayers()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40044/ --- (Updated Nov. 9, 2015, 4:01 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40044: Remove unused parameter in moveLayers()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40044/#review105771 --- Ship it! Ship It! - Timothy Chen On Nov. 7, 2015, 1:29 a.m., Gi

Re: Review Request 39987: [3/5] Added 'Master::authorize(Un)reserveResources()' for Reserve/Unreserve.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39987/ --- (Updated Nov. 9, 2015, 11:42 p.m.) Review request for mesos, Adam B, Jie Yu, Mi

Re: Review Request 39043: Added support for HTTP Authentication in Mesos.

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

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/#review105765 --- src/slave/slave.hpp (line 679)

Re: Review Request 39285: Added Quota Request Validation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39285/ --- (Updated Nov. 9, 2015, 11:30 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 35711: Disallow special characters in role name.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35711/#review105756 --- include/mesos/roles.hpp (lines 40 - 41)

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38059/ --- (Updated Nov. 9, 2015, 11:14 p.m.) Review request for mesos, Bernd Mathiske, Jo

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38059/ --- (Updated Nov. 9, 2015, 11:11 p.m.) Review request for mesos, Bernd Mathiske, Jo

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 Nov. 9, 2015, 11:10 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 40102: Windows: Unified POSIX and Windows implementation of process.hpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40102/#review105762 --- Patch looks great! Reviews applied: [39537, 39538, 39539, 39540, 3

Re: Review Request 40107: Removed unused checks in command executor.

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

Re: Review Request 39223: Added Quota Request Validation Tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39223/ --- (Updated Nov. 9, 2015, 10:45 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 39043: Added support for HTTP Authentication in Mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39043/ --- (Updated Nov. 9, 2015, 11:38 p.m.) Review request for mesos, Adam B, Bernd Math

Re: Review Request 38950: Http Authenticators can be loaded as modules from mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38950/ --- (Updated Nov. 9, 2015, 11:38 p.m.) Review request for mesos, Adam B, Bernd Math

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 Nov. 9, 2015, 11:33 p.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 38000: Added an API for libprocess users to interact with http::AuthenticatorManager

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38000/ --- (Updated Nov. 9, 2015, 11:31 p.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 37999: Implemented http::AuthenticatorManager

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

Re: Review Request 40102: Windows: Unified POSIX and Windows implementation of process.hpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40102/#review105759 --- Bad patch! Reviews applied: [39537, 39538, 39539, 39540, 39541, 39

Re: Review Request 39949: Document and simplify libprocess initialization synchronization logic.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39949/ --- (Updated Nov. 9, 2015, 2:23 p.m.) Review request for mesos, Benjamin Hindman, J

Re: Review Request 39948: Remove some undocumented, commented-out code within libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39948/ --- (Updated Nov. 9, 2015, 2:23 p.m.) Review request for mesos, Benjamin Hindman an

Re: Review Request 39420: Added '--parent' option and made apply-review.sh call apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39420/#review105755 --- Can you split this into 2 reviews? one adding --parent option and t

Re: Review Request 39410: Added support for github to apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39410/#review105728 --- Ship it! support/apply-reviews.py (line 42)

Re: Review Request 39401: Quota: Updated allocate() in the hierarchical allocator to support quota.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39401/ --- (Updated Nov. 9, 2015, 10:01 p.m.) Review request for mesos, Bernd Mathiske, Jo

Re: Review Request 39860: Quota: Replaced "slave" with "agent" in allocator logs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39860/ --- (Updated Nov. 9, 2015, 9:59 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 40107: Removed unused checks in command executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40107/#review105749 --- Ship it! Ship It! - Kapil Arya On Nov. 9, 2015, 4:37 p.m., Timo

Review Request 40102: Windows: Unified POSIX and Windows implementation of process.hpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40102/ --- Review request for mesos, Artem Harutyunyan, Michael Hopcroft, Joris Van Remoort

Re: Review Request 40107: Removed unused checks in command executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40107/#review105747 --- Ship it! Ship It! - Jie Yu On Nov. 9, 2015, 9:37 p.m., Timothy

Re: Review Request 40107: Removed unused checks in command executor.

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

Review Request 40107: Removed unused checks in command executor.

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

Re: Review Request 39285: Added Quota Request Validation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39285/ --- (Updated Nov. 9, 2015, 8:25 p.m.) Review request for mesos, Alexander Rukletsov

Re: Review Request 38747: Adding digest utilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38747/ --- (Updated Nov. 9, 2015, 8:17 p.m.) Review request for mesos, Ben Mahler, Gilbert

Re: Review Request 39949: Document and simplify libprocess initialization synchronization logic.

> On Nov. 7, 2015, 7:35 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/process.cpp, lines 742-770 > > > > > > Could we document why we're not just using process::Once to clean this > > up? > > > > The

Re: Review Request 39331: Support docker local store pull image simultaneously

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

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

> On Nov. 5, 2015, 12:24 p.m., Joseph Wu wrote: > > src/master/quota_handler.cpp, lines 83-88 > > > > > > Why don't you just parse a QuotaInfo object instead of a > > form-serialized body (with JSON components)? > >

Re: Review Request 39285: Added Quota Request Validation.

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

Re: Review Request 40062: MESOS-2315 Deprecate/Remove CommandInfo::ContainerInfo

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40062/#review105723 --- include/mesos/mesos.proto

Re: Review Request 39331: Support docker local store pull image simultaneously

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39331/ --- (Updated Nov. 9, 2015, 11:26 a.m.) Review request for mesos, Anand Mazumdar, Ji

Re: Review Request 39989: [5/5] Added framework authorization for dynamic reservation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39989/#review105714 --- Patch looks great! Reviews applied: [39985, 39986, 39987, 39988, 3

Re: Review Request 39966: Fix a typo in ssl.md: After -> Before

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39966/#review105712 --- Ship it! Ship It! - Joris Van Remoortere On Nov. 5, 2015, 4:02

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

> On Nov. 2, 2015, 8:38 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/mkdtemp.hpp, > > lines 34-71 > > > > > > Can you please explain why we can't just use `mktemp`

Re: Review Request 39559: Windows: Implemented `os::mkdtemp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39559/ --- (Updated Nov. 9, 2015, 5:58 p.m.) Review request for mesos, Artem Harutyunyan,

Re: Review Request 38956: Quota: Added allocator-agnostic tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38956/ --- (Updated Nov. 9, 2015, 5:49 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 38956: Quota: Added allocator-agnostic tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38956/ --- (Updated Nov. 9, 2015, 5:39 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 38110: Quota: Checked sanity of quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38110/ --- (Updated Nov. 9, 2015, 5:37 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 39989: [5/5] Added framework authorization for dynamic reservation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39989/ --- (Updated Nov. 9, 2015, 5:26 p.m.) Review request for mesos, Adam B, Jie Yu, Mic

Re: Review Request 39986: [2/5] Enabled the Authorizer to handle Reserve/Unreserve ACLs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39986/ --- (Updated Nov. 9, 2015, 5:26 p.m.) Review request for mesos, Adam B, Jie Yu, Mic

Re: Review Request 39988: [4/5] Added authorization for dynamic reservation master endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39988/ --- (Updated Nov. 9, 2015, 5:26 p.m.) Review request for mesos, Adam B, Jie Yu, Mic

Re: Review Request 39987: [3/5] Added 'Master::authorize' for Reserve/Unreserve.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39987/ --- (Updated Nov. 9, 2015, 5:26 p.m.) Review request for mesos, Adam B, Jie Yu, Mic

Re: Review Request 39985: [1/5] Introduced ACL protobuf definitions for dynamic reservation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39985/ --- (Updated Nov. 9, 2015, 5:25 p.m.) Review request for mesos, Adam B, Jie Yu, Mic

Re: Review Request 39018: Added JSON parsing for Resources.

> On Nov. 9, 2015, 6:43 a.m., Adam B wrote: > > src/common/resources.cpp, lines 356-357 > > > > > > Doesn't Resources::validate() return an Error? Why not use that Error > > and its message with your return? De

Re: Review Request 40087: Added a list of protocols supported by the Mesos fetcher as requested in MESOS-2783

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40087/#review105693 --- docs/fetcher.md (line 19)

Re: Review Request 39018: Added JSON parsing for Resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39018/ --- (Updated Nov. 9, 2015, 4:48 p.m.) Review request for mesos, Adam B, Alexander R

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38059/ --- (Updated Nov. 9, 2015, 4:39 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 40087: Added a list of protocols supported by the Mesos fetcher as requested in MESOS-2783

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

Review Request 40087: Added a list of protocols supported by the Mesos fetcher as requested in MESOS-2783

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40087/ --- Review request for mesos, Alexander Rojas and Bernd Mathiske. Bugs: MESOS-2783

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 Nov. 9, 2015, 4:06 p.m.) Review request for mesos, Alexander Rukletsov

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38059/ --- (Updated Nov. 9, 2015, 3:59 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 39420: Added '--parent' option and made apply-review.sh call apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39420/#review105685 --- Bad patch! Reviews applied: [38705, 38883, 39410] Failed command:

Re: Review Request 39410: Added support for github to apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39410/ --- (Updated Nov. 9, 2015, 7:32 a.m.) Review request for mesos, Adam B, Joris Van R

Re: Review Request 39420: Added '--parent' option and made apply-review.sh call apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39420/ --- (Updated Nov. 9, 2015, 7:31 a.m.) Review request for mesos, Joris Van Remoorter

Re: Review Request 39420: Added '--parent' option and made apply-review.sh call apply-reviews.py.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39420/ --- (Updated Nov. 9, 2015, 7:31 a.m.) Review request for mesos, Joris Van Remoorter

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38059/#review105192 --- src/master/quota_handler.cpp (lines 44 - 47)

Re: Review Request 38218: Quota: Extended the Allocator interface with quota-related methods.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38218/ --- (Updated Nov. 9, 2015, 3:25 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 39317: Quota: Moved `QuotaInfo` protobuf into a separate package.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39317/ --- (Updated Nov. 9, 2015, 3:23 p.m.) Review request for mesos, Bernd Mathiske, Joe

Re: Review Request 38059: Quota: Created a bare pipeline for quota set requests.

> On Oct. 7, 2015, 7:41 a.m., Bernd Mathiske wrote: > > src/master/quota_handler.cpp, line 66 > > > > > > Given that we have the request object, we can output more diagnostic > > info here. > > Guangya Liu wrote: >

  1   2   >