Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/#review172279 --- Ship it! Ship It! - Vinod Kone On April 14, 2017, 9:14

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/ --- (Updated April 14, 2017, 9:14 p.m.) Review request for mesos, Adam B,

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/ --- (Updated April 14, 2017, 3:51 p.m.) Review request for mesos, Adam B,

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/#review171917 --- src/authorizer/local/authorizer.cpp Lines 323-327 (original),

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/ --- (Updated April 13, 2017, 6:56 p.m.) Review request for mesos, Adam B,

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/#review171722 --- Ship it! Ship It! - Alexander Rojas On April 12, 2017, 9:31

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/ --- (Updated April 12, 2017, 7:31 a.m.) Review request for mesos, Adam B,

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

> On April 11, 2017, 12:27 a.m., Vinod Kone wrote: > > src/authorizer/local/authorizer.cpp > > Lines 725 (patched) > > > > > > Is this based on the assumption that claims subjects only come from > > executors and

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

> On April 11, 2017, 12:27 a.m., Vinod Kone wrote: > > src/authorizer/local/authorizer.cpp > > Lines 725 (patched) > > > > > > Is this based on the assumption that claims subjects only come from > > executors and

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/#review171506 --- include/mesos/authorizer/authorizer.proto Line 176 (original),

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

> On April 7, 2017, 11:40 a.m., Alexander Rojas wrote: > > src/authorizer/local/authorizer.cpp > > Lines 654-657 (original), 683-690 (patched) > > > > > > I'm not so sure returning a `RejectingObjectApprover()` is

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/ --- (Updated April 7, 2017, 11:25 p.m.) Review request for mesos, Adam B,

Re: Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/#review171320 --- src/authorizer/local/authorizer.cpp Line 576 (original), 601-603

Review Request 58254: Added implicit executor authorization to the agent operator API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58254/ --- Review request for mesos, Adam B, Alexander Rojas, Till Toenshoff, and Vinod