> On Dec. 6, 2016, 1:55 a.m., Vinod Kone wrote: > > src/tests/api_tests.cpp, lines 3728-3793 > > <https://reviews.apache.org/r/54381/diff/1/?file=1576457#file1576457line3728> > > > > why did you change these tests? the idea with the tests was to verify > > that `Containerizer::attach()` was getting called from the API handler.
Because one needs to find the containers to perform authorization, and with these tests there was no container to be found, so no authz to be performed and therefor not mockContainerizer to be called. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54381/#review158090 ----------------------------------------------------------- On Dec. 5, 2016, 5:19 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54381/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2016, 5:19 p.m.) > > > Review request for mesos, Adam B, Kevin Klues, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Introduces authorization support for the newly introduced debug API > calls: `AttachContainerInput` and `AttachContainerOutput`. > > > Diffs > ----- > > src/slave/http.cpp 91eebe117d3dc86dd5b1fd47156c01e147165cef > src/slave/slave.hpp 4b94dff9c1f9f212f84984674268ef38b43d93bd > src/tests/api_tests.cpp 7c70f284a9225ca23abc7049f48f3efba314b641 > > Diff: https://reviews.apache.org/r/54381/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rojas > >
