FlakyTest: testEmptyLedgerLosesQuorumEventually

Open liangyepianzhou opened this issue 2 months ago • 2 comments

Motivation

Fix flaky test: https://github.com/apache/bookkeeper/actions/runs/18525197327/job/52795009929?pr=4652

Changes

To maintain consistency and reduce future modifications, all assertion timeouts in testEmptyLedgerLosesQuorumEventually that use the format assertTrue(xxx, latch.await(xx)) have been unified to 20 seconds.

These assertion timeouts in this test have been changed several times, with each change only increasing the timeout of one assertion. To avoid repeated modifications in the future, these timeouts in this test have been aligned to the longest timeout value that was previously set.

https://github.com/apache/bookkeeper/blob/bde9ff2503ae71c4ef03b212577a1a8be673123a/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/BookieAutoRecoveryTest.java#L418

Oct 15 '25 11:10 liangyepianzhou

/bkbot rerun

Oct 24 '25 09:10 liangyepianzhou

reopen due to https://github.com/apache/bookkeeper/pull/4665

Nov 04 '25 06:11 liangyepianzhou