Re: Review Request 52766: Fix a bug in insufficient successes during initial_interval_secs

2016-10-12 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52766/#review152423 --- We reverted the related code so I marked this diff as discarded.

Re: Review Request 52766: Fix a bug in insufficient successes during initial_interval_secs

2016-10-11 Thread Kai Huang
> On Oct. 12, 2016, 1:49 a.m., David McLaughlin wrote: > > src/test/python/apache/aurora/executor/common/test_health_checker.py, line > > 500 > > > > > > Maybe I'm misunderstanding something - but this test suite se

Re: Review Request 52766: Fix a bug in insufficient successes during initial_interval_secs

2016-10-11 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52766/#review152269 --- Ship it! Master (e9abb22) is green with this patch. ./build-s

Re: Review Request 52766: Fix a bug in insufficient successes during initial_interval_secs

2016-10-11 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52766/ --- (Updated Oct. 12, 2016, 5:01 a.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 52766: Fix a bug in insufficient successes during initial_interval_secs

2016-10-11 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52766/#review152252 --- src/test/python/apache/aurora/executor/common/test_health_checker

Re: Review Request 52766: Fix a bug in insufficient successes during initial_interval_secs

2016-10-11 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52766/#review152248 --- Ship it! Master (e9abb22) is green with this patch. ./build-s