Skip to content

test: improve docker compose retry for transient network failures#3507

Merged
tippmar-nr merged 1 commit intomainfrom
test/flaky-container-test-docker-network
Mar 26, 2026
Merged

test: improve docker compose retry for transient network failures#3507
tippmar-nr merged 1 commit intomainfrom
test/flaky-container-test-docker-network

Conversation

@tippmar-nr
Copy link
Copy Markdown
Member

@tippmar-nr tippmar-nr commented Mar 26, 2026

Summary

  • Increase docker compose startup retries from 1 to 2
  • Add 5-second delay between cleanup and retry to let Docker fully release network resources

🤖 Generated with Claude Code

Increase max retries from 1 to 2 and add a 5-second delay between
cleanup and retry to give Docker time to fully release network
resources. Addresses flaky LinuxUnicodeSpecialCharactersTest failures
where docker compose up gets "network not found" due to Docker daemon
race conditions when multiple compose projects are active.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tippmar-nr tippmar-nr marked this pull request as ready for review March 26, 2026 14:59
@tippmar-nr tippmar-nr requested a review from a team as a code owner March 26, 2026 14:59
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.77%. Comparing base (d63e16a) to head (fe5011c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3507      +/-   ##
==========================================
- Coverage   81.79%   81.77%   -0.02%     
==========================================
  Files         508      508              
  Lines       34220    34220              
  Branches     4040     4040              
==========================================
- Hits        27990    27984       -6     
- Misses       5265     5269       +4     
- Partials      965      967       +2     
Flag Coverage Δ
Agent 82.76% <ø> (-0.02%) ⬇️
Profiler 71.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tippmar-nr tippmar-nr enabled auto-merge (squash) March 26, 2026 15:27
@tippmar-nr tippmar-nr merged commit 83412d5 into main Mar 26, 2026
118 checks passed
@tippmar-nr tippmar-nr deleted the test/flaky-container-test-docker-network branch March 26, 2026 15:29
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