gharris1727 opened a new pull request, #12791: URL: https://github.com/apache/kafka/pull/12791
Signed-off-by: Greg Harris <greg.har...@aiven.io> There is existing flakiness in this test even with the previous patch to address flakiness https://github.com/apache/kafka/pull/11871 This should resolve the observed flakiness because the MockTime will only progress time when the test interacts with it. One small change to the test was necessary: adding an explicit sleep within the retried operation in testNoBackoffTimeAndFail. Because the no-backoff case does not call the mockTime sleep, the test as-written would enter into an infinite retry loop, with every retry happening on the same mockTime timestamp. The test needed to be modified such that the retried operation advances the clock slightly to prevent this. Additionally, revert the previous flakiness patch to restore the original 100ms timeouts, down from 10 seconds. This should be backported to every branch that the previous flakiness fix was applied to: 2.6, 2.7, 2.8, 3.0, 3.1, 3.2, 3.3. Thanks! ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
