Skip to content

fix(ci): recover relay opens and isolate VM proxy ports - #3618

Closed
drew wants to merge 1 commit into
mainfrom
codex/stability-shortlist-relay-base
Closed

drew wants to merge 1 commit into
mainfrom
codex/stability-shortlist-relay-base

Conversation

@drew

@drew drew commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Recover relay opens after supervisor reconnect and prevent VM proxy fixture port collisions. Clean restack of #3583.

Related Issue

No issue required: localized fixes for the Docker sync and VM proxy failures cited in #3583.

Changes

  • Replay unclaimed relay opens after each accepted supervisor session.
  • Give VM proxy fixtures distinct ports and allow the host supervisor its required /proc access.
  • Document reconnect replay behavior.

Testing

Checklist

  • Conventional, signed-off commit.
  • Architecture documentation updated.

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew drew added the automated-flake-find PRs produced by the automated E2E flake-finding campaign label Sep 23, 2026
@copy-pr-bot

copy-pr-bot Bot commented Sep 23, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@drew
drew added this pull request to stack #3623 September 23, 2026 15:16
@drew drew closed this Sep 28, 2026
@elezar

elezar commented Oct 1, 2026

Copy link
Copy Markdown
Member

Cross-reference: #4011 has been approved and submitted to the merge queue. It covers the pending-relay reconnect recovery originally implemented in #3583 and restacked here, together with duplicate-delivery suppression from #3588/#3619. It adds atomic enqueue/session validation, a shared setup deadline, SSH proxy shutdown/error propagation, and bounded Docker diagnostics.

Draft #3766 remains the follow-up for deterministic reconnect conformance coverage: rebase it after #4011 merges and remove the overlapping server fix while retaining the tests and architecture documentation. The VM fixture changes here are separate from #4011, and the underlying Docker startup stall remains unresolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-flake-find PRs produced by the automated E2E flake-finding campaign

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants