Skip to content

test(conformance): cover relay reconnect recovery - #3766

Draft
elezar wants to merge 2 commits into
mainfrom
codex/relay-reconnect-conformance
Draft

elezar wants to merge 2 commits into
mainfrom
codex/relay-reconnect-conformance

Conversation

@elezar

@elezar elezar commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Summary

Add deterministic conformance coverage for relay attachment recovery across supervisor-session startup and reconnect windows. Carry the focused server fix in this draft so the regression scenario demonstrates both the failure and the intended recovery behavior.

Stack

Main stack: #3641 → #3597 → #3637.

This draft is a side branch from #3597 and should remain draft until the relay fix is accepted independently of the file-transfer migration.

Related Issue

No issue required: this is a localized regression test and fix for the intermittent relay loss originally investigated in #3583 and #3588.

Failure evidence

Changes

  • split sandbox lifecycle coverage into independent state-transition, relay-readiness, and relay-reconnect scenarios
  • exercise startup relay readiness in bounded asynchronous pairs
  • inject a control-session disconnect for Docker, rootful Podman, and rootless Podman without pausing the workload
  • replay pending relay opens when the prior supervisor session was removed before reconnection
  • suppress duplicate relay delivery to the same supervisor session
  • document reconnect replay behavior

Testing

  • mise run pre-commit
  • Focused Docker tmachine scenario reproduced relay open timed out before the fix
  • Focused rootful Podman scenario reproduced relay open timed out before the fix
  • Rootless Podman scenario reproduced relay open timed out before the fix
  • Docker, rootful Podman, and rootless Podman conformance suites passed with the fix applied before the rebase
  • Post-rebase GitHub E2E checks

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture documentation updated

@copy-pr-bot

copy-pr-bot Bot commented Sep 28, 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.

@elezar
elezar force-pushed the codex/file-transfer-conformance branch from 3e3f907 to 9caaf25 Compare September 28, 2026 07:29
@elezar
elezar force-pushed the codex/relay-reconnect-conformance branch from aaa901f to 45a097e Compare September 28, 2026 07:30
@elezar
elezar force-pushed the codex/file-transfer-conformance branch from 9caaf25 to 893e2d2 Compare September 28, 2026 08:22
@elezar
elezar force-pushed the codex/relay-reconnect-conformance branch from 45a097e to 4d9636e Compare September 28, 2026 08:23
@elezar
elezar force-pushed the codex/file-transfer-conformance branch from 893e2d2 to f679ce1 Compare September 28, 2026 08:46
@elezar
elezar force-pushed the codex/relay-reconnect-conformance branch from 4d9636e to 022b7e1 Compare September 28, 2026 08:46
Base automatically changed from codex/file-transfer-conformance to main September 28, 2026 09:13
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar force-pushed the codex/relay-reconnect-conformance branch from 022b7e1 to c39fa9a Compare September 28, 2026 11:37

This branch has not been deployed

No deployments
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.

1 participant