Skip to content

[fix][test] Stabilize WebService rate limiting test#25866

Merged
lhotari merged 1 commit into
apache:masterfrom
oneby-wang:fix-webservice-rate-limiting-flake
May 25, 2026
Merged

[fix][test] Stabilize WebService rate limiting test#25866
lhotari merged 1 commit into
apache:masterfrom
oneby-wang:fix-webservice-rate-limiting-flake

Conversation

@oneby-wang
Copy link
Copy Markdown
Contributor

Motivation

WebServiceTest.testRateLimiting can be flaky because setup admin HTTP requests and lookup HTTP requests share the same broker HTTP rate limiter. With a 10 QPS limiter, requests sent too closely together can consume freshly restored permits and cause an unexpected 429 before the test reaches the burst section.

Modifications

  • Compute the wait interval from rateLimit and use it for lookup pacing.
  • Add the same short wait between setup admin calls and after namespace creation.

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

@lhotari lhotari added this to the 5.0.0-M1 milestone May 25, 2026
@lhotari lhotari merged commit 3b6a760 into apache:master May 25, 2026
43 checks passed
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.

2 participants