Fix flaky "preserves failures" tests #8817
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These 2 tests have been increasingly flaky lately. At first I thought it was some bug that was introduced, but then I went back to 2.0.x and 2.1-RC1 and it was flaky back then as well when the number of test executions was increased.
I believe that the flakiness was due to a weird combination of
interruptible
within regions that should be strictly uninterruptible (e.g., withinensuring
). I haven't managed to get the tests to fail after these changes