Skip to content

[fix][test] Fix flaky ClusterMigrationTest.testClusterMigrationWithReplicationBacklog#25344

Closed
merlimat wants to merge 1 commit intoapache:masterfrom
merlimat:fix/flaky-ClusterMigrationTest
Closed

[fix][test] Fix flaky ClusterMigrationTest.testClusterMigrationWithReplicationBacklog#25344
merlimat wants to merge 1 commit intoapache:masterfrom
merlimat:fix/flaky-ClusterMigrationTest

Conversation

@merlimat
Copy link
Contributor

@merlimat merlimat commented Mar 17, 2026

Flaky test failure

java.lang.AssertionError: expected [false] but found [true]
	at org.testng.Assert.fail(Assert.java:110)
	at org.testng.Assert.failNotEquals(Assert.java:1577)
	at org.testng.Assert.assertFalse(Assert.java:78)
	at org.testng.Assert.assertFalse(Assert.java:88)
	at org.apache.pulsar.broker.service.ClusterMigrationTest.testClusterMigrationWithReplicationBacklog(ClusterMigrationTest.java:525)

Summary

  • Fix flaky ClusterMigrationTest.testClusterMigrationWithReplicationBacklog by increasing the retry timeout for the replication backlog drain check from 10s to 30s.
  • After restarting broker3, the replication backlog may take longer to drain due to reconnection and replication catchup time.

Documentation

  • doc-not-needed
    (Your PR doesn't need any doc update)

Matching PR in forked repository

No response

Tip

Add the labels ready-to-test and area/test to trigger the CI.

…plicationBacklog

After restarting broker3, the replication backlog may take longer
than 10s to drain. Increase the retry timeout from 10 to 30 retries
to allow sufficient time for the backlog to clear.
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 17, 2026
@merlimat merlimat closed this Mar 17, 2026
@merlimat merlimat deleted the fix/flaky-ClusterMigrationTest branch March 17, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant