Skip to content

Conversation

leocavalhere6
Copy link

@leocavalhere6 leocavalhere6 commented Sep 25, 2025

Fixes #21875

Main Issue: #21875

Motivation

The integration test ExtensibleLoadManagerTest.startBroker was flaky due to a timeout
when waiting for all brokers to become active and for topics to be created and looked up.
This caused intermittent CI failures.

Modifications

  • Increased Awaitility timeout from 120s to 4 minutes.
  • Separated broker readiness check from topic creation and lookup.
  • Ensured all brokers are active before creating partitioned topics.
  • Retained original logic for topic creation and lookup without affecting existing tests.

Verifying this change

  • Verified locally: the flaky test now consistently passes.
  • No new tests added; existing integration tests cover the modified logic.

Does this pull request potentially affect one of the following parts

  • Dependencies
  • Public API
  • Schema
  • Default configuration values
  • Threading model
  • Binary protocol
  • REST endpoints
  • Admin CLI options
  • Metrics
  • Deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: N/A

Copy link

@leocavalhere6 Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky-test: ExtensibleLoadManagerTest.startBroker
1 participant