----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36927/#review93549 -----------------------------------------------------------
Patch looks great! Reviews applied: [36927] All tests passed. - Mesos ReviewBot On July 29, 2015, 11:52 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36927/ > ----------------------------------------------------------- > > (Updated July 29, 2015, 11:52 p.m.) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Framework authorization is now done through `authorizeFramework` which is > consistent with `authorizeTask`. > Also, `validateFrameworkAuthentication` captures the validation for > authentication. > > I also made registration and re-registration consistent: > > * Both perform the check for root submission, rather than just registration. > * Authentication checks in `_registerFramework` and `_reregisterFramework` > are now comprehensive (thanks to `validateFrameworkAuthentication`). > > > Diffs > ----- > > src/master/master.hpp 1135049d34f8ed4d56c795ad28c9ca1d07237b63 > src/master/master.cpp 2bfec2f69375444925252480142ee409b8474761 > > Diff: https://reviews.apache.org/r/36927/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >
