Skip to content

test(e2e): run podman suite with tmachine - #3637

Merged
elezar merged 3 commits into
mainfrom
codex/tmachine-podman-e2e
Sep 28, 2026
Merged

elezar merged 3 commits into
mainfrom
codex/tmachine-podman-e2e

Conversation

@elezar

@elezar elezar commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Summary

Migrate the portable Rust Podman E2E coverage from the legacy wrapper approach in #2982 to a dedicated tmachine test archive. Keep this coverage separate from the focused driver-podman user-namespace suite and schedule both under driver-specific integration tests.

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 #3597 for relay reconnect coverage.

Related Issue

No issue required: this is a focused test-infrastructure migration that supersedes draft PR #2982.

Merge after #3597, which replaces the removed e2e/rust/tests/sync.rs coverage with portable file-transfer conformance scenarios.

Changes

  • Build an e2e-podman nextest archive from e2e/rust with the e2e-podman feature
  • Add an independent tmachine e2e-podman testsuite and rootless Podman playbook
  • Keep the existing driver-podman suite focused on default, auto, keep-id, and private UID-map equivalence
  • Remove the superseded podman_userns E2E binary now covered by driver-podman
  • Remove sync.rs after its file-transfer cases move to conformance in test(conformance): migrate file transfer coverage #3597
  • Run both suites independently in the driver-specific integration matrix
  • Exclude wrapper-coupled or locally failing test binaries through an explicit follow-up list
  • Port the portable sparse-policy fixture and bounded Podman teardown changes from test(e2e): add rootless podman nix vm lane #2982
  • Document local archive creation and tmachine execution

Testing

  • mise run pre-commit passes
  • nix run .#build-artifacts-test-archives
  • nix run .#tmachine -- test fedora-podman-rootless binaries e2e-podman (43 passed, 2 skipped)
  • nix run .#tmachine -- test fedora-podman-rootless binaries driver-podman (default, auto, keep-id, and private profiles passed)
  • E2E tests added/updated

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

@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 d209c3b. 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.

drew
drew previously approved these changes Sep 23, 2026
SDAChess
SDAChess previously approved these changes Sep 28, 2026

@SDAChess SDAChess left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. Maybe as a follow-up we can consider cleaning up the testsuite running with a nextest role + renaming e2e to something more meaningful like podman-driver or something.

@elezar

elezar commented Sep 28, 2026

Copy link
Copy Markdown
Member Author

This looks good. Maybe as a follow-up we can consider cleaning up the testsuite running with a nextest role + renaming e2e to something more meaningful like podman-driver or something.

Yes. The e2e name is intentional as I wanted to add coverage and follow-up with a more focussed cleanup of the tests. The podman-specific tests should move to the existing podman-driver suite / crate.

Base automatically changed from codex/file-transfer-conformance to main September 28, 2026 09:13
@elezar
elezar force-pushed the codex/tmachine-podman-e2e branch from c42ffca to 860421d Compare September 28, 2026 09:13
@elezar
elezar force-pushed the codex/tmachine-podman-e2e branch from 860421d to 894dd6e Compare September 28, 2026 10:08
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar force-pushed the codex/tmachine-podman-e2e branch from 894dd6e to bf94eae Compare September 28, 2026 10:11
@elezar
elezar added this pull request to the merge queue Sep 28, 2026
Merged via the queue into main with commit eef8bec Sep 28, 2026
166 of 168 checks passed
@elezar
elezar deleted the codex/tmachine-podman-e2e branch September 28, 2026 11:26
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.

3 participants