Skip to content

Fix stale gRPC endpoints after app-first startup retries - #1896

Merged
WhitWaldo merged 2 commits into
masterfrom
copilot/investigate-integration-tests-failure-again
Aug 16, 2026
Merged

WhitWaldo merged 2 commits into
masterfrom
copilot/investigate-integration-tests-failure-again

Conversation

Copilot AI commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Description

Workflow integration tests intermittently connected to a disposed sidecar after an app-first startup retry, surfacing as transient gRPC Unavailable and Connection refused failures.

  • Root cause
    • Port properties preferred the previous DaprdContainer ports over newly assigned retry overrides.
  • Change
    • Explicit HTTP and gRPC overrides now take precedence over stale container mappings.
    • Startup-order coverage verifies refreshed ports remain visible after sidecar startup.

Issue reference

No linked issue.

Checklist

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Co-authored-by: WhitWaldo <2238529+WhitWaldo@users.noreply.github.com>
@codecov

codecov Bot commented Aug 16, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.62%. Comparing base (18a82b5) to head (5dd7e3f).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/Dapr.Testcontainers/Harnesses/BaseHarness.cs 0.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1896      +/-   ##
==========================================
- Coverage   62.64%   62.62%   -0.03%     
==========================================
  Files         338      338              
  Lines        9965     9965              
  Branches     1171     1171              
==========================================
- Hits         6243     6241       -2     
+ Misses       3401     3400       -1     
- Partials      321      324       +3     
Flag Coverage Δ
net10.0 62.57% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: WhitWaldo <2238529+WhitWaldo@users.noreply.github.com>
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.

2 participants