----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69397/#review211211 -----------------------------------------------------------
Ship it! Ship It! - Benjamin Mahler On Dec. 11, 2018, 1:48 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69397/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2018, 1:48 p.m.) > > > Review request for mesos, Benjamin Mahler and Meng Zhu. > > > Bugs: MESOS-9469 > https://issues.apache.org/jira/browse/MESOS-9469 > > > Repository: mesos > > > Description > ------- > > This patch replaces uses of `framework::validateRoles` with the more > high-level `framework::validate`. Since e.g., the master invokes > `validate` instead of directly invoking `validateRoles` this ensures > that our tests more closely match master behavior, and helps > preventing that we test code which is not invoked in reality. > > > Diffs > ----- > > src/tests/master_validation_tests.cpp > 3aa7b2a2dcaf054ad2a67c52462493c97b7178c4 > > > Diff: https://reviews.apache.org/r/69397/diff/2/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > >
