Skip to content

test(conformance): migrate file transfer coverage - #3597

Merged
elezar merged 3 commits into
mainfrom
codex/file-transfer-conformance
Sep 28, 2026
Merged

elezar merged 3 commits into
mainfrom
codex/file-transfer-conformance

Conversation

@elezar

@elezar elezar commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Summary

Move driver-agnostic CLI file-transfer coverage from the Docker Rust E2E suite into reusable conformance scenarios that run against each conformance compute environment.

Stack

  1. test(cli): migrate gateway-free smoke coverage #3641 — gateway-free CLI smoke coverage
  2. test(conformance): migrate file transfer coverage #3597 — file-transfer conformance coverage
  3. test(e2e): run podman suite with tmachine #3637 — Podman E2E tmachine archive

Draft follow-up #3766 branches from this PR for relay reconnect coverage.

Related Issue

No issue required: this is a localized test-suite migration with no user-facing behavior change.

Merge after #3641, which moves gateway-free smoke coverage out of the legacy Rust E2E suite. Relay reconnect regression coverage and its focused fix were split into draft PR #3766.

Changes

  • replace e2e/rust/tests/sync.rs with portable file-transfer/{round-trip,git-filtering,path-safety} scenarios
  • exercise directory, large-file, and single-file round trips
  • verify Git-aware upload filtering and explicit unfiltered fallback behavior
  • verify download workspace boundaries and safe handling of special basenames
  • install Git in tmachine conformance guests and preserve all test failures
  • update conformance architecture guidance

Testing

  • mise run pre-commit
  • cargo test -p openshell-conformance (18 passed)
  • Docker tmachine file-transfer scenarios passed before the rebase
  • Rootful Podman tmachine file-transfer scenarios passed before the rebase
  • Rootless Podman tmachine file-transfer scenarios passed before the rebase
  • Post-rebase GitHub E2E checks

Checklist

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

@elezar elezar added the test:e2e Requires end-to-end coverage label Sep 23, 2026
@github-actions

Copy link
Copy Markdown

Label test:e2e applied for bc8109d. Open the existing run and click Re-run all jobs to execute with the label set. The run will execute the standard E2E suite after building the required gateway, sandbox, and supervisor images once. The matching required CI gate status on this PR will flip green automatically once the run finishes.

@elezar

elezar commented Sep 23, 2026

Copy link
Copy Markdown
Member Author

/ok to test bc8109d

@elezar
elezar force-pushed the codex/file-transfer-conformance branch 3 times, most recently from d40b060 to fbde8df Compare September 23, 2026 14:28
@elezar
elezar force-pushed the codex/file-transfer-conformance branch 2 times, most recently from 8f32455 to 3e3f907 Compare September 28, 2026 07:22
@elezar
elezar force-pushed the codex/file-transfer-conformance branch from 3e3f907 to 9caaf25 Compare September 28, 2026 07:29
@elezar
elezar enabled auto-merge September 28, 2026 08:14
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar force-pushed the codex/file-transfer-conformance branch from 9caaf25 to 893e2d2 Compare September 28, 2026 08:22
@elezar
elezar changed the base branch from main to gateway-free-cli-tests/elezar September 28, 2026 08:23
stack merge was automatically disabled September 28, 2026 08:23

Merge commits are not allowed on this repository

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar added this pull request to stack #3770 September 28, 2026 08:44
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar force-pushed the codex/file-transfer-conformance branch from 893e2d2 to f679ce1 Compare September 28, 2026 08:46
Merged via the queue into main with commit 0c29d8e Sep 28, 2026
105 checks passed
Base automatically changed from gateway-free-cli-tests/elezar to main September 28, 2026 09:13
@elezar
elezar deleted the codex/file-transfer-conformance branch September 28, 2026 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:e2e Requires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants