----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26796/#review56968 -----------------------------------------------------------
Patch looks great! Reviews applied: [26674, 26071, 26796] All tests passed. - Mesos ReviewBot On Oct. 16, 2014, 4:57 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26796/ > ----------------------------------------------------------- > > (Updated Oct. 16, 2014, 4:57 p.m.) > > > Review request for mesos, Niklas Nielsen and Till Toenshoff. > > > Repository: mesos-git > > > Description > ------- > > This allows the user to specify the modules to loaded during tests. The > modules are made available to the master and slave subsystems. > > > Diffs > ----- > > src/tests/flags.hpp 189fad9a8125aa8f76a7abadc330a7f0ec7cc337 > src/tests/main.cpp 90a7ddcb5695d4c9a7760e18285548d99e7bcfa8 > src/tests/module.hpp PRE-CREATION > src/tests/module.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/26796/diff/ > > > Testing > ------- > > Tested by hand using the following commandline: > ./bin/mesos-tests.sh --modules="/path/to/modules.json" > > > Thanks, > > Kapil Arya > >
