Re: Review Request 47222: Add authorization to /quota endpoint.

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

Re: Review Request 47222: Add authorization to /quota endpoint.

2016-05-13 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47222/#review133093 --- Let's punt on the coarse-grained authz for now. - Alexander Rukle

Re: Review Request 47222: Add authorization to /quota endpoint.

2016-05-12 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47222/ --- (Updated May 12, 2016, 4:21 p.m.) Review request for mesos, Adam B and Alexande

Re: Review Request 47222: Add authorization to /quota endpoint.

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

Re: Review Request 47222: Add authorization to /quota endpoint.

2016-05-10 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47222/#review132609 --- src/master/quota_handler.cpp (line 491)

Review Request 47222: Add authorization to /quota endpoint.

2016-05-10 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47222/ --- Review request for mesos, Adam B and Alexander Rukletsov. Bugs: MESOS-5336