----------------------------------------------------------- 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), 323-327 (patched) <https://reviews.apache.org/r/58254/#comment244914> Populate for LAUNCH_NESTED_CONTAINER_SESSION Add authorization test for this (enable authZ be default in all default executor tests) src/authorizer/local/authorizer.cpp Line 659 (original), 726-733 (patched) <https://reviews.apache.org/r/58254/#comment244918> Comment explaining the purpose of this conditional. - Greg Mann On April 13, 2017, 6:56 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > 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, Alexander Rojas, Till Toenshoff, and Vinod > Kone. > > > Bugs: MESOS-7014 > https://issues.apache.org/jira/browse/MESOS-7014 > > > Repository: mesos > > > Description > ------- > > This patch updates the agent handlers for the LAUNCH_, WAIT_, > and KILL_NESTED_CONTAINER calls of the operator API to set the > `container_id` field within the authorization object, > facilitating implicit executor authorization. > > > Diffs > ----- > > include/mesos/authorizer/authorizer.proto > 736f76d552956f2351ffd40fc51d088dff83f8c8 > src/authorizer/local/authorizer.cpp > 1c1f912794cfe61112a0e513b217ba3a755f35f1 > src/slave/http.cpp 468cf332d79ed7315ecf51955235735dec0a2df1 > > > Diff: https://reviews.apache.org/r/58254/diff/4/ > > > Testing > ------- > > Testing details can be found at the end of this chain. > > > Thanks, > > Greg Mann > >
