geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Owen Nichols <>
Subject Re: [vote/discuss]Override stressNewTest for Pull Request #4250?
Date Thu, 31 Oct 2019 22:07:46 GMT
I’ve retriggered StressNew <>
with a temporarily-increased timeout of 12 hours so we can see how long it would actually
take, to have some data point whether to propose a permanent timeout increase or whether breaking
up into multiple PRs is should be the standard way to get around this.

> On Oct 31, 2019, at 2:52 PM, Donal Evans <> wrote:
> +1 to allowing this PR to be merged, although I'd lean strongly toward
> facilitating this by temporarily increasing the timeout on the job to allow
> it to actually pass rather than a manual override of the StressNewTest.
> The fact that it's passed over 7000 times without failing is pretty strong
> evidence that it's not a flaky test, which is what StressNewTest is
> supposed to catch, so there doesn't seem to be any risk associated with
> circumventing it in this case, but if there's a feasible solution that
> doesn't involve "cheating" or ignoring the test job, then that would be
> preferable.
> - Donal
> On Thu, Oct 31, 2019 at 2:04 PM Jason Huynh <> wrote:
>> Greetings,
>> We have a pull request ( that is
>> running into a problem with stressNewTest.  Mostly the tests that are being
>> run are RollingUpgrade tests that take quite a bit of time to run the full
>> suite.  Because these tests are added/modified, the stressNewTest doesn't
>> have enough time to complete the run because it runs them N(50) number of
>> times.
>> However what has completed is 7400 tests and none of them have failed:
>> We would like to get this fix in before branching the next release, but are
>> unable to due to stressNewTest gating the merge button.  I know we have
>> another thread about overrides etc, and maybe this is a data point, but
>> this isn't meant to discuss that.
>> Would everyone be able to agree to allow someone to manually override and
>> merge this commit in (title of PR and reviews pending)?

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message