-
Notifications
You must be signed in to change notification settings - Fork 360
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix test_ts_of_lsn_api flakiness #7599
Conversation
2904 tests run: 2783 passed, 0 failed, 121 skipped (full report)Code coverage* (full report)
* collected from Rust tests only The comment gets automatically updated with the latest test results
5abba19 at 2024-05-06T16:49:28.991Z :recycle: |
Baseline case, no patches (50 retries): one occurence. |
Doubling the wait time seems to have fixed it. The test now takes 30% longer but that's fine, still below 20 seconds per run. report link. |
Turning on synchronous commit still makes it occur (300 retries): one occurence. |
report after doubling the time again and enabling synchronous commit. Not a single failure. I'm not happy about this fix because it just changes parameters around, and likely the test is still going to exhibit flakiness, but it's solving the immediate problem I'd say. |
For posterity, this is the link to the log that triggered the regression. |
Changes parameters to fix the flakiness of `test_ts_of_lsn_api`. Already now, the amount of flakiness of the test is pretty low. With this, it's even lower. cc #5768
Changes parameters to fix the flakiness of
test_ts_of_lsn_api
. Already now, the amount of flakiness of the test is pretty low. With this, it's even lower.cc #5768