Skip to content

[Internal] Tests: Fixes circuit breaker replication delay for CI reliability#5714

Merged
NaluTripician merged 2 commits into
masterfrom
users/nalutripician/test-fix-circuit-breaker-delay
Mar 24, 2026
Merged

[Internal] Tests: Fixes circuit breaker replication delay for CI reliability#5714
NaluTripician merged 2 commits into
masterfrom
users/nalutripician/test-fix-circuit-breaker-delay

Conversation

@NaluTripician
Copy link
Copy Markdown
Contributor

Summary

Increases the replication delay from 3s to 5s in two circuit breaker integration tests to account for slower multi-region data propagation on CI networks.

Root Cause

Both tests create items and then wait for replication before running circuit breaker scenarios. The 3000ms delay was insufficient on slow CI networks, causing data to not be fully replicated when assertions ran.

Tests Fixed

Test Failures (30 days) Pass Rate
ReadItemAsync_WithCircuitBreakerEnabled...ThirdRegion 9 97.99%
ReadItemAsync_WithCircuitBreakerDisabled...Override 2 99.55%

Impact

…reliability

Increases replication delay from 3s to 5s in circuit breaker
integration tests to account for slower multi-region data
propagation on CI networks.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good!

@NaluTripician NaluTripician changed the title [Internal] Tests: Increases circuit breaker replication delay for CI reliability [Internal] Tests: Fixes circuit breaker replication delay for CI reliability Mar 19, 2026
@NaluTripician NaluTripician merged commit c63af81 into master Mar 24, 2026
32 checks passed
@NaluTripician NaluTripician deleted the users/nalutripician/test-fix-circuit-breaker-delay branch March 24, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants