merlimat opened a new pull request, #25355: URL: https://github.com/apache/pulsar/pull/25355
## Flaky test failure ``` TopicTerminationTest.testCreatingProducerTasksCleanupWhenOnTerminatedTopic:123 » ConditionTimeout Assertion condition expected [0] but found [6] within 10 seconds. ``` ## Summary - Fix flaky `TopicTerminationTest.testCreatingProducerTasksCleanupWhenOnTerminatedTopic` which asserts `timer.pendingTimeouts() == 0` but the `HashedWheelTimer` is shared across all tests in the suite via `SharedPulsarCluster`. - Other producers/consumers from previous tests may have pending timeouts unrelated to this test, causing the assertion to find 6 pending timeouts instead of 0. - Fix by recording the baseline `pendingTimeouts` before the failed producer creation and asserting the count returns to that baseline, rather than expecting absolute zero. - Also add the missing `@Test` annotation — the method was only running when targeted by surefire `-Dtest=` pattern matching. ## Documentation - [x] `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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
