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

2016-06-27 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/#review139569 --- src/tests/authorization_tests.cpp (line 2108)

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 p.m

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

2016-06-06 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/ --- (Updated June 6, 2016, 3:11 p.m.) Review request for mesos, Adam B, Till Toensh

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

2016-06-06 Thread Joerg Schad
> On June 6, 2016, 2:47 p.m., Vinod Kone wrote: > > src/tests/authorization_tests.cpp, line 2106 > > > > > > what do you mean by "is only allowed for `ANY`"? > > > > did you mean "matches an ACL whose Objec

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

2016-06-06 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/#review136259 --- src/tests/authorization_tests.cpp (line 2106)

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

2016-06-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/#review136229 --- Patch looks great! Reviews applied: [48264] Passed command: expo

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

2016-06-06 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/ --- Review request for mesos, Adam B, Till Toenshoff, and Vinod Kone. Bugs: MESOS-5