----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45372/#review125628 -----------------------------------------------------------
FWIW, there is als this very old review requests that is talking about the same tests https://reviews.apache.org/r/31380/diff/1#index_header. What does Brian mean with "calling .converge"? - Stephan Erb On March 28, 2016, 5:41 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45372/ > ----------------------------------------------------------- > > (Updated March 28, 2016, 5:41 a.m.) > > > Review request for Aurora, John Sirois and Zameer Manji. > > > Repository: aurora > > > Description > ------- > > Posting this patch to at least start a conversation on fixing this test; i've > noticed it flaking pretty frequently lately. Here i take the quick and dirty > approach of removing the sleep and glossing over the `total_latency_secs` > value. > > > Diffs > ----- > > src/test/python/apache/aurora/executor/common/test_health_checker.py > 19c4f76347e34374c29974c182d1f4c118bcb18d > > Diff: https://reviews.apache.org/r/45372/diff/ > > > Testing > ------- > > None yet > > > Thanks, > > Bill Farner > >
