Skip to content

Conversation

@brycekahle
Copy link
Member

What does this PR do?

Adds configuration for connection dial error limit and per-dial error timeout in seconds.

Which scenarios this will impact?

aws/ec2
aws/microVMs
azure
gcp
hyperv

Motivation

In some situations including local microvms, waiting 8 minutes (500s) for failure is too long.

Additional Notes

@brycekahle brycekahle requested review from a team as code owners October 24, 2025 00:38
@brycekahle brycekahle force-pushed the bryce.kahle/configure-connection-retry branch from 3d80307 to 9f0858f Compare October 27, 2025 22:16
@brycekahle
Copy link
Member Author

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Oct 28, 2025

View all feedbacks in Devflow UI.

2025-10-28 14:59:13 UTC ℹ️ Start processing command /merge


2025-10-28 14:59:19 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 49m (p90).


2025-10-28 15:41:07 UTCMergeQueue: The checks failed on this merge request

Tests failed on this commit 0b27190:

What to do next?

  • Investigate the failures and when ready, re-add your pull request to the queue!
  • If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
  • Any question, go check the FAQ.

@brycekahle
Copy link
Member Author

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Oct 28, 2025

View all feedbacks in Devflow UI.

2025-10-28 17:33:16 UTC ℹ️ Start processing command /merge


2025-10-28 17:33:22 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 49m (p90).


2025-10-28 18:30:05 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit fd3f8f4 into main Oct 28, 2025
10 checks passed
@dd-mergequeue dd-mergequeue bot deleted the bryce.kahle/configure-connection-retry branch October 28, 2025 18:30
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.

4 participants