Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-08-06 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/#review49770 --- Ship it! Ship It! - Vinod Kone On July 30, 2014, 9:14 p.m., Timo

Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/#review49198 --- Patch looks great! Reviews applied: [23772] All tests passed. - M

Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-30 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/ --- (Updated July 30, 2014, 9:14 p.m.) Review request for mesos, Niklas Nielsen and

Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-30 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/ --- (Updated July 30, 2014, 9:12 p.m.) Review request for mesos and Vinod Kone. C

Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-21 Thread Timothy Chen
> On July 22, 2014, 1:42 a.m., Vinod Kone wrote: > > src/tests/health_check_tests.cpp, line 213 > > > > > > Sorry, didn't understand the description. What is the bug? Basically the test expects 4 health failures and t

Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/#review48326 --- src/tests/health_check_tests.cpp

Re: Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-21 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/ --- (Updated July 21, 2014, 9:10 p.m.) Review request for mesos and Vinod Kone. B

Review Request 23772: Re-enable HealthCheckTest.ConsecutiveFailures test

2014-07-21 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23772/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1613 https://issues.ap