Re: Review Request 37785: Fix Flaky SlaveTest.HTTPSchedulerSlaveRestart test

2015-09-01 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37785/#review97353 --- Ship it! Ship It! - Vinod Kone On Aug. 26, 2015, 3:07 a.m., Anan

Re: Review Request 37785: Fix Flaky SlaveTest.HTTPSchedulerSlaveRestart test

2015-08-31 Thread Anand Mazumdar
> On Aug. 31, 2015, 11:55 p.m., Vinod Kone wrote: > > src/tests/slave_tests.cpp, lines 2624-2625 > > > > > > If resume the clock here, the slave can still retry registration right? > > You probably want to just pau

Re: Review Request 37785: Fix Flaky SlaveTest.HTTPSchedulerSlaveRestart test

2015-08-31 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37785/#review97206 --- src/tests/slave_tests.cpp (lines 2624 - 2625)

Re: Review Request 37785: Fix Flaky SlaveTest.HTTPSchedulerSlaveRestart test

2015-08-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37785/#review96491 --- Patch looks great! Reviews applied: [37785] All tests passed. - M