----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31315/#review73659 -----------------------------------------------------------
Ship it! src/tests/master_slave_reconciliation_tests.cpp <https://reviews.apache.org/r/31315/#comment120055> How about: ``` // Before we obtain the metrics, ensure that the master finishes // processesing the status update and increments its Counters. Clock::pause(); Clock::settle(); ``` - Ben Mahler On Feb. 23, 2015, 7:56 p.m., Dominic Hamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31315/ > ----------------------------------------------------------- > > (Updated Feb. 23, 2015, 7:56 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2366 > https://issues.apache.org/jira/browse/MESOS-2366 > > > Repository: mesos > > > Description > ------- > > Wait for status acknowledgement before getting master metrics > > > Diffs > ----- > > src/tests/master_slave_reconciliation_tests.cpp > 230f5293f4906e09eb55db128df4a19b1e308428 > > Diff: https://reviews.apache.org/r/31315/diff/ > > > Testing > ------- > > 300 repeats without failure of the test under a debugger. > > > Thanks, > > Dominic Hamon > >
