Re: Review Request 47123: Added --modules_dir flag to read module manifests from a directory.

2016-05-23 Thread Till Toenshoff
eed to keep stuff in sync? Same for all other references of that flag / help messages? - Till Toenshoff On May 20, 2016, 2:40 a.m., Kapil Arya wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 47786: Removed unneeded #includes from type_utils.hpp.

2016-05-25 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47786/#review134728 --- Ship it! Ship It! - Till Toenshoff On May 24, 2016, 6:06

Re: Review Request 47753: Enabled building libprocess without building 3rdparty first.

2016-05-25 Thread Till Toenshoff
running a `bootstrap` I am getting ``` configure.ac:62: error: required file '3rdparty/Makefile.in' not found configure.ac:64: error: required file '3rdparty/gmock_sources.cc.in' not found ``` - Till Toenshoff On May 24, 2016, 2:49 a.

Re: Review Request 47610: Reverted adding libelf as a default dynamic library in stout.

2016-05-25 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47610/#review134749 --- Ship it! Ship It! - Till Toenshoff On May 20, 2016, 12:56

Re: Review Request 47609: Reverted adding libelf as a default dynamic library in libprocess.

2016-05-25 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47609/#review134750 --- Ship it! Ship It! - Till Toenshoff On May 19, 2016, 6:08

Re: Review Request 47821: Remove SASL dependency for Windows builds.

2016-05-25 Thread Till Toenshoff
n do a much better job than I ever could for such updated comment :) Please note that we commonly have the "link" to the JIRA issue as trailing sentence; "... See MESOS-." - Till Toenshoff On May 25, 2016, 6:4

Re: Review Request 47515: Enforced a constraint on `DiskInfo.Persistence.principal`.

2016-05-26 Thread Till Toenshoff
Greg for taking the time and typing in the comments as discussed verbally - much appreciated :). - Till Toenshoff On May 23, 2016, 4:37 a.m., Greg Mann wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-26 Thread Till Toenshoff
Greg for taking the time and typing in the comments as discussed verbally - much appreciated :). - Till Toenshoff On May 25, 2016, 6:08 p.m., Greg Mann wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 47519: Updated an example framework to specify its principal.

2016-05-26 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47519/#review135091 --- Ship it! Ship It! - Till Toenshoff On May 18, 2016, 8:04

Re: Review Request 47520: Updated test helpers with creator principal.

2016-05-26 Thread Till Toenshoff
Greg for taking the time and typing in the comments as discussed verbally - much appreciated :). - Till Toenshoff On May 18, 2016, 8:04 a.m., Greg Mann wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 47521: Added creator principal to '/create-volumes' tests.

2016-05-26 Thread Till Toenshoff
Greg for taking the time and typing in the comments as discussed verbally - much appreciated :). - Till Toenshoff On May 18, 2016, 8:05 a.m., Greg Mann wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 47522: Added creator principal to persistent volume tests.

2016-05-26 Thread Till Toenshoff
Greg for taking the time and typing in the comments as discussed verbally - much appreciated :). - Till Toenshoff On May 18, 2016, 8:06 a.m., Greg Mann wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 47516: Added creator principal to tests.

2016-05-26 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47516/#review135096 --- Ship it! Ship It! - Till Toenshoff On May 18, 2016, 8:06

Re: Review Request 48033: Fixed signed/unsigned comparison introduced in `f22af98`.

2016-05-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48033/#review135492 --- Ship it! Ship It! - Till Toenshoff On May 30, 2016, 10:23

Re: Review Request 48034: Added v1 files to CMakeLists.txt.

2016-05-30 Thread Till Toenshoff
this was 0ba45c8c0af084bd8a2278170f9ab0a8467d631d (https://reviews.apache.org/r/48010) - Till Toenshoff On May 30, 2016, 10:21 a.m., Benjamin Bannier wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 48039: Updated `QuotaHandler` to only one authorization request per action.

2016-05-30 Thread Till Toenshoff
tps://reviews.apache.org/r/48039/#comment200501> Is there a JIRA around introducing LocalAuthorizer specific deprecation cycles - I would like to understand why we are not pushing this request out to any authorizer (module). - Till Toenshoff On May 30, 2016, 1:17 p.m., Michael Park

Re: Review Request 47906: Added/updated documentation for --modules and --modules_dir flags.

2016-05-30 Thread Till Toenshoff
906/#comment200503> This looks borked. - Till Toenshoff On May 26, 2016, 5:04 p.m., Kapil Arya wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 47905: Added test for --modules_dir flag.

2016-05-30 Thread Till Toenshoff
/#comment200506> Missing? ``` #include ``` ``` #include ``` - Till Toenshoff On May 26, 2016, 5:57 p.m., Kapil Arya wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

Re: Review Request 47123: Added --modules_dir flag to read module manifests from a directory.

2016-05-30 Thread Till Toenshoff
creating a local const in the first place? src/tests/main.cpp (line 31) <https://reviews.apache.org/r/47123/#comment200509> Insert blank line. - Till Toenshoff On May 26, 2016, 5:04 p.m., Kapil Arya wrote: > > --- >

Re: Review Request 47906: Added/updated documentation for --modules and --modules_dir flags.

2016-05-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47906/#review135519 --- Ship it! Ship It! - Till Toenshoff On May 30, 2016, 2:43

Re: Review Request 48048: Added documentation for access_sandboxes and access_mesos_logs acls.

2016-05-30 Thread Till Toenshoff
g/r/48048/#comment200542> s/whoe/whose docs/authorization.md (line 132) <https://reviews.apache.org/r/48048/#comment200545> I have trouble understanding ``` _It still takes types ANY and NONE to allow or deny access._ ``` Why "still" - in wh

Re: Review Request 47505: Updated authorizer.proto Subject, Object and Request.

2016-05-31 Thread Till Toenshoff
/mesos/authorizer/authorizer.proto 0142476 Diff: https://reviews.apache.org/r/47505/diff/ Testing --- make check (OSX) Thanks, Till Toenshoff

Re: Review Request 47505: Updated authorizer.proto Subject, Object and Request.

2016-05-31 Thread Till Toenshoff
reply, visit: https://reviews.apache.org/r/47505/#review133707 ------- On May 31, 2016, 5:59 p.m., Till Toenshoff wrote: > > --- > This is an automatically

Re: Review Request 47505: Updated authorizer.proto Subject, Object and Request.

2016-05-31 Thread Till Toenshoff
to upgrade the messages in the future. Yup, follow-up coming. - Till --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47505/#review133707 ------- On May 31, 2016, 5:59 p.m., Till Toenshoff wrote: >

Re: Review Request 47505: Updated authorizer.proto Subject, Object and Request.

2016-05-31 Thread Till Toenshoff
://reviews.apache.org/r/47505/diff/ Testing --- make check (OSX) Thanks, Till Toenshoff

Re: Review Request 47505: Updated authorizer.proto Subject, Object and Request.

2016-05-31 Thread Till Toenshoff
/authorizer.proto 0142476a2f9c905aafd3e41adc9842445c9309e7 Diff: https://reviews.apache.org/r/47505/diff/ Testing --- make check (OSX) Thanks, Till Toenshoff

Review Request 48085: Added sanity checks to local authorizer.

2016-05-31 Thread Till Toenshoff
Vinod Kone. Repository: mesos Description --- see summary. Diffs - src/authorizer/local/authorizer.cpp 29bcb404c390bfb446af11796836a4a8181bbeee Diff: https://reviews.apache.org/r/48085/diff/ Testing --- make check Thanks, Till Toenshoff

Review Request 48093: Updated comments in authorizer.proto.

2016-05-31 Thread Till Toenshoff
Schad, and Vinod Kone. Repository: mesos Description --- Updated comments in authorizer.proto. Diffs - include/mesos/authorizer/authorizer.proto 0142476a2f9c905aafd3e41adc9842445c9309e7 Diff: https://reviews.apache.org/r/48093/diff/ Testing --- Thanks, Till Toenshoff

Re: Review Request 48085: Added sanity checks to local authorizer.

2016-05-31 Thread Till Toenshoff
/ Testing --- make check Thanks, Till Toenshoff

Re: Review Request 48093: Updated comments in authorizer.proto.

2016-05-31 Thread Till Toenshoff
----- On May 31, 2016, 9:15 p.m., Till Toenshoff wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48093/ >

Re: Review Request 48132: Updated mesos-style to add 3rdparty/stout.

2016-06-01 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48132/#review135806 --- Ship it! Ship It! - Till Toenshoff On June 1, 2016, 4:01

Re: Review Request 48133: Fixed a minor style issue in stout.

2016-06-01 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48133/#review135807 --- Ship it! Ship It! - Till Toenshoff On June 1, 2016, 4:14

Review Request 48137: Fixed reserve resources authorization test.

2016-06-01 Thread Till Toenshoff
Description --- see summary. Diffs - src/tests/authorization_tests.cpp bcad250b4376ba71b04e5280ec6e4cd17abd8ae6 Diff: https://reviews.apache.org/r/48137/diff/ Testing --- make check Thanks, Till Toenshoff

Re: Review Request 48141: Added --batch-mode to mvn invocation.

2016-06-01 Thread Till Toenshoff
141/#comment200893> Can we be sure that all Mesos supported versions of Maven actually support this flag? - Till Toenshoff On June 1, 2016, 8:21 p.m., Kapil Arya wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 48164: Suppress all warnings inside googlemock.

2016-06-02 Thread Till Toenshoff
workaround Benjamin! src/Makefile.am (line 2086) <https://reviews.apache.org/r/48164/#comment200928> Let's add a comment explaining this move, please. - Till Toenshoff On June 2, 2016, 9:31 a.m., Benjamin Bannier wrote: > > -

Re: Review Request 48164: Suppress all warnings inside googlemock.

2016-06-02 Thread Till Toenshoff
> On June 2, 2016, 9:36 a.m., Till Toenshoff wrote: > > Thanks for testing a supplying a neat workaround Benjamin! s/a supplying/and supplying/ - Till --- This is an automatically generated e-mail. To reply, vis

Re: Review Request 48164: Suppress all warnings inside googlemock.

2016-06-02 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48164/#review135936 --- Ship it! Ship It! - Till Toenshoff On June 2, 2016, 12:13

Re: Review Request 47930: Stout: Fixed broken libelf dependency.

2016-06-04 Thread Till Toenshoff
apil Arya wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47930/ > --- > > (Updated May 26, 2016, 11:30 p.m.) > > > Review request for mesos and Till To

Re: Review Request 47755: Libprocess: `make tests` now automatically builds dependencies.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47755/#review136188 --- Ship it! Ship It! - Till Toenshoff On May 24, 2016, 2:49

Re: Review Request 47756: Stout: `make tests` now automatically builds dependencies.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47756/#review136189 --- Ship it! Ship It! - Till Toenshoff On May 24, 2016, 2:49

Re: Review Request 47924: Added AM_CONDITIONALS: STANDALONE_{LIBPROCES, STOUT}.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47924/#review136190 --- Ship it! Ship It! - Till Toenshoff On May 26, 2016, 11:30

Re: Review Request 47925: Libprocess: Fixed a typo in Makefile.am.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47925/#review136191 --- Ship it! Ship It! - Till Toenshoff On May 26, 2016, 11:30

Re: Review Request 47928: Libprocess: Prepare for standalone compilation.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47928/#review136192 --- Ship it! Ship It! - Till Toenshoff On June 4, 2016, 7:04

Re: Review Request 47929: Libprocess: made standalone build work again.

2016-06-04 Thread Till Toenshoff
so, how does that work with a non-bundled parser? Finally, why are we calling this "libry_http_parser" and not "libhttp_parser"? - Till Toenshoff On May 26, 2016, 11:30 p.m., Kapil Arya wrote: > > ---

Re: Review Request 47931: Stout: Prepare for standalone compilation.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47931/#review136194 --- Ship it! Ship It! - Till Toenshoff On June 4, 2016, 7:01

Re: Review Request 47932: Stout: made standalone build work again.

2016-06-04 Thread Till Toenshoff
pache.org/r/47932/#comment201209> Wrong header again, use the "Apache License Version 2.0" header instead. - Till Toenshoff On May 26, 2016, 11:30 p.m., Kapil Arya wrote: > > --- > This is an automaticall

Re: Review Request 47929: Libprocess: made standalone build work again.

2016-06-04 Thread Till Toenshoff
.org/r/47929/#comment201211> Wrong header, use the "Apache License Version 2.0" header. - Till Toenshoff On May 26, 2016, 11:30 p.m., Kapil Arya wrote: > > --- > This is an automatically generated

Re: Review Request 48246: Added configure check for separate fts library to libprocess.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48246/#review136198 --- Ship it! Ship It! - Till Toenshoff On June 4, 2016, 2:30

Re: Review Request 48245: Added configure check for separate fts library to stout.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48245/#review136199 --- Ship it! Ship It! - Till Toenshoff On June 4, 2016, 2:30

Re: Review Request 48247: Added configure check for separate fts library to Mesos.

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48247/#review136197 --- Ship it! Ship It! - Till Toenshoff On June 4, 2016, 2:31

Re: Review Request 48244: Removed support for parsing the filesystem table (fstab).

2016-06-04 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48244/#review136200 --- Ship it! - Till Toenshoff On June 4, 2016, 2:29 a.m., Neil

Re: Review Request 48244: Removed support for parsing the filesystem table (fstab).

2016-06-04 Thread Till Toenshoff
244/#comment201212> I was too slow when trying to commit this. Needs rebase now, sorry. - Till Toenshoff On June 4, 2016, 2:29 a.m., Neil Conway wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 48263: Fixed documentation for MESOS-5405.

2016-06-07 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48263/#review136425 --- Ship it! Ship It! - Till Toenshoff On June 6, 2016, 8:22

Re: Review Request 48263: Fixed documentation for MESOS-5405.

2016-06-07 Thread Till Toenshoff
il. To reply, visit: > https://reviews.apache.org/r/48263/ > --- > > (Updated June 6, 2016, 8:22 a.m.) > > > Review request for mesos, Adam B, Till Toenshoff, and Vinod Kone. > > > Bugs: MESOS-5405 > http

Re: Review Request 48264: Added test for optional request.object field.

2016-06-07 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/#review136426 --- Ship it! Ship It! - Till Toenshoff On June 6, 2016, 3:11

Re: Review Request 48269: Removed unused imports from protobuf files.

2016-06-08 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48269/#review136632 --- Ship it! Ship It! - Till Toenshoff On June 6, 2016, 10:24

Re: Review Request 48409: Fixed broken links in the docs.

2016-06-13 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48409/#review137269 --- Ship it! Ship It! - Till Toenshoff On June 8, 2016, 1:59

Re: Review Request 48420: Added some links to the docs.

2016-06-13 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48420/#review137270 --- Ship it! Ship It! - Till Toenshoff On June 8, 2016, 4:58

Re: Review Request 48626: Fixed typo in markdown styleguide.

2016-06-13 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48626/#review137271 --- Ship it! Ship It! - Till Toenshoff On June 13, 2016, 9:54

Re: Review Request 48598: Added changes to upgraded.md.

2016-06-13 Thread Till Toenshoff
tps://reviews.apache.org/r/48598/#comment202429> Would it make sense to link the authorization documentation here? - Till Toenshoff On June 13, 2016, 9:50 a.m., Joerg Schad wrote: > > --- > This is an automatically generated e

Re: Review Request 47374: Separated mesos test helpers into a separate library.

2016-06-13 Thread Till Toenshoff
12:03 a.m.) > > > Review request for mesos, Adam B, Artem Harutyunyan, Jie Yu, Kapil Arya, Jan > Schlicht, and Till Toenshoff. > > > Repository: mesos > > > Description > --- > > This gives external projects easier access to the test

Re: Review Request 47374: Separated mesos test helpers into a separate library.

2016-06-13 Thread Till Toenshoff
971 - 1980) <https://reviews.apache.org/r/47374/#comment202408> Good point, I very much agree. - Till Toenshoff On June 11, 2016, 12:03 a.m., Joseph Wu wrote: > > --- > This is an automatically generated e-mail. To reply, visi

Re: Review Request 48206: Fix example in configuration docs.

2016-06-14 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48206/#review137645 --- Ship it! Ship It! - Till Toenshoff On June 3, 2016, 2:31

Re: Review Request 47374: Separated mesos test helpers into a separate library.

2016-06-15 Thread Till Toenshoff
> On June 14, 2016, 12:26 a.m., Till Toenshoff wrote: > > Overall, I am not convinced this is reaching the goals we have just yet... > > > > Let me try to specify our goals; > > (a) we want to enable mesos-modules (tests) to reuse our test helpers/utils > >

Re: Review Request 48495: Added missing 'get_weights' actions to authorization.md.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48495/#review137971 --- Ship it! Ship It! - Till Toenshoff On June 15, 2016, 7:53

Re: Review Request 48496: Fixed misleading acls example in authorization.md.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48496/#review137972 --- Ship it! Ship It! - Till Toenshoff On June 15, 2016, 6:50

Re: Review Request 48497: Added documentation on starting to use acls.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48497/#review137977 --- Ship it! Ship It! - Till Toenshoff On June 16, 2016, 1:44

Re: Review Request 48765: Added 'labels' and 'discovery' to generated 'ExecutorInfo'.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48765/#review137989 --- Ship it! Ship It! - Till Toenshoff On June 16, 2016, 12:56

Re: Review Request 48765: Added 'labels' and 'discovery' to generated 'ExecutorInfo'.

2016-06-16 Thread Till Toenshoff
> On June 16, 2016, 3 p.m., Till Toenshoff wrote: > > Ship It! Will fix the tiny comment typo while committing. - Till --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48765/#rev

Re: Review Request 48790: Added note about generation of `ExecutorInfo` for `CommandInfo`.

2016-06-16 Thread Till Toenshoff
-framework-development-guide.md (line 312) <https://reviews.apache.org/r/48790/#comment203239> s/by/on/ - Till Toenshoff On June 16, 2016, 1:01 p.m., Joerg Schad wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 48789: Added tests for sandbox authorization.

2016-06-16 Thread Till Toenshoff
.org/r/48789/#comment203236> one space less src/tests/authorization_tests.cpp (line 2242) <https://reviews.apache.org/r/48789/#comment203238> s/a// src/tests/authorization_tests.cpp (line 2243) <https://reviews.apache.org/r/48789/#comment203237> s/awithout/without/

Re: Review Request 48806: Added Berlin usergroup to community page.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48806/#review138099 --- Ship it! Ship It! - Till Toenshoff On June 16, 2016, 6 p.m

Re: Review Request 48781: Marked some optional fields in state.json as required.

2016-06-16 Thread Till Toenshoff
--- > > (Updated June 16, 2016, 10:06 a.m.) > > > Review request for mesos, Joerg Schad and Till Toenshoff. > > > Bugs: MESOS-5588 > https://issues.apache.org/jira/browse/MESOS-5588 > > > Repository: mesos > > > Description

Re: Review Request 48764: Refactored sandbox authorization logic to use ObjectAuthorizer.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48764/#review138119 --- Ship it! Ship It! - Till Toenshoff On June 16, 2016, 5:34

Re: Review Request 48635: Fixed various typos and grammar nits in upgrade notes.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48635/#review138123 --- Ship it! Ship It! - Till Toenshoff On June 13, 2016, 1:16

Re: Review Request 48684: Revised documentation for task reconciliation.

2016-06-16 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48684/#review138126 --- Ship it! Ship It! - Till Toenshoff On June 14, 2016, 12:03

Re: Review Request 48781: Marked some optional fields in acls.proto as required.

2016-06-17 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48781/#review138198 --- Ship it! Ship It! - Till Toenshoff On June 17, 2016, 9:58

Re: Review Request 48849: Removed capturing this in sandbox authorization.

2016-06-17 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48849/#review138239 --- Ship it! Ship It! - Till Toenshoff On June 17, 2016, 2:22

Review Request 48919: Updated HTTPProxy to return a failure message in the HTTP result.

2016-06-18 Thread Till Toenshoff
/48919/diff/ Testing --- make check & functional testing. Thanks, Till Toenshoff

Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-18 Thread Till Toenshoff
/ Testing --- make check & functional testing. Thanks, Till Toenshoff

Review Request 48920: Updated the HTTP result returned by failures of authn/authz.

2016-06-18 Thread Till Toenshoff
703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48920/diff/ Testing --- make check & functional testing. Thanks, Till Toenshoff

Review Request 48923: Updated test to expect ServiceUnavailable.

2016-06-19 Thread Till Toenshoff
- src/tests/slave_tests.cpp ad36b74c12e52435b356bced1d15bd33983949e0 Diff: https://reviews.apache.org/r/48923/diff/ Testing --- make check & functional testing. Thanks, Till Toenshoff

Re: Review Request 48925: Implement GET_WEIGHTS Call in v1 master API.

2016-06-19 Thread Till Toenshoff
(lines 94 - 95) <https://reviews.apache.org/r/48925/#comment203624> Remove the whitespaces here please - then, to make it fit into 80 chars, break at the argument. ``` response.mutable_get_weights()->add_weight_infos()->CopyFrom( weightInfo); ``` - Till To

Review Request 48932: Updated CHANGELOG to include note about '503' on authorizer failures.

2016-06-19 Thread Till Toenshoff
/ Testing --- Thanks, Till Toenshoff

Re: Review Request 48919: Updated HTTPProxy to return a failure message in the HTTP result.

2016-06-19 Thread Till Toenshoff
reg Mann, and Kapil Arya. > > > Bugs: MESOS-5637 > https://issues.apache.org/jira/browse/MESOS-5637 > > > Repository: mesos > > > Description > --- > > see summary. > > > Diffs > - > > 3rdparty/libprocess/src/process.cpp > 703f673a98102958c5e2b0c1833efad2ddc53ef8 > > Diff: https://reviews.apache.org/r/48919/diff/ > > > Testing > --- > > make check & functional testing. > > > Thanks, > > Till Toenshoff > >

Re: Review Request 48932: Updated CHANGELOG to include note about '503' on authorizer failures.

2016-06-19 Thread Till Toenshoff
thorizer fails to process the request. Like it - ty! - Till --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48932/#review138493 -----

Re: Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-19 Thread Till Toenshoff
.apache.org/r/48918/#review138488 --- On June 19, 2016, 3:02 a.m., Till Toenshoff wrote: > > --- > This is an automatically generated e-mail. To reply, v

Re: Review Request 48937: Added a TODO about possible security issues due to misspelled ACLs.

2016-06-19 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48937/#review138509 --- Ship it! Ship It! - Till Toenshoff On June 20, 2016, 12:12

Re: Review Request 48919: Updated HTTPProxy to return a failure message in the HTTP result.

2016-06-19 Thread Till Toenshoff
/libprocess/src/process.cpp 703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48919/diff/ Testing --- make check & functional testing. Thanks, Till Toenshoff

Re: Review Request 48920: Updated the HTTP result returned by failures of authn/authz.

2016-06-19 Thread Till Toenshoff
the problem. Diffs (updated) - 3rdparty/libprocess/src/process.cpp 703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48920/diff/ Testing --- make check & functional testing. Thanks, Till Toenshoff

Re: Review Request 48932: Updated CHANGELOG to include note about '503' on authorizer failures.

2016-06-19 Thread Till Toenshoff
aed56300f1090c816ab2893c20e87fd37230a9a7 Diff: https://reviews.apache.org/r/48932/diff/ Testing --- Thanks, Till Toenshoff

Re: Review Request 48919: Updated HTTPProxy to return a failure message in the HTTP result.

2016-06-19 Thread Till Toenshoff
703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48919/diff/ Testing (updated) --- make check (OSX and some Linux distros) Thanks, Till Toenshoff

Re: Review Request 48920: Updated the HTTP result returned by failures of authn/authz.

2016-06-19 Thread Till Toenshoff
/libprocess/src/process.cpp 703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48920/diff/ Testing (updated) --- make check (OSX and some Linux distros) & functional testing. Thanks, Till Toenshoff

Re: Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-19 Thread Till Toenshoff
://reviews.apache.org/r/48918/diff/ Testing (updated) --- make check (OSX and some Linux distros) Thanks, Till Toenshoff

Re: Review Request 48923: Updated test to expect ServiceUnavailable.

2016-06-19 Thread Till Toenshoff
will now receive a ServiceUnavailable. Diffs - src/tests/slave_tests.cpp ad36b74c12e52435b356bced1d15bd33983949e0 Diff: https://reviews.apache.org/r/48923/diff/ Testing (updated) --- make check (OSX and some Linux distros) Thanks, Till Toenshoff

Re: Review Request 48936: Explained why fields in the acts.Entity must be required.

2016-06-20 Thread Till Toenshoff
e here to make that more obvious. How about: ``` NOTE: When adding new actions to these proto definitions that entities in these messages are `required`. Absent ... ``` - Till Toenshoff On June 20, 2016, 12:12 a.m., Alexander Rukletsov

Re: Review Request 48920: Updated the HTTP result returned by failures of authn/authz.

2016-06-20 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48920/#review138547 ------- On June 20, 2016, 5:49 a.m., Till Toenshoff wrote: > > -

Re: Review Request 48919: Updated HTTPProxy to return a failure message in the HTTP result.

2016-06-20 Thread Till Toenshoff
/libprocess/src/process.cpp 703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48919/diff/ Testing --- make check (OSX and some Linux distros) Thanks, Till Toenshoff

Re: Review Request 48920: Updated the HTTP result returned by failures of authn/authz.

2016-06-20 Thread Till Toenshoff
the problem. Diffs (updated) - 3rdparty/libprocess/src/process.cpp 703f673a98102958c5e2b0c1833efad2ddc53ef8 Diff: https://reviews.apache.org/r/48920/diff/ Testing --- make check (OSX and some Linux distros) & functional testing. Thanks, Till Toenshoff

Re: Review Request 48932: Updated CHANGELOG to include note about '503' on authorizer failures.

2016-06-20 Thread Till Toenshoff
aed56300f1090c816ab2893c20e87fd37230a9a7 Diff: https://reviews.apache.org/r/48932/diff/ Testing --- Thanks, Till Toenshoff

<    4   5   6   7   8   9   10   11   12   13   >