Skip to content

test(e2e): skip docker gpu test in rust suite#1103

Merged
pimlock merged 1 commit into
mainfrom
test/skip-docker-gpu-rust-e2e
May 1, 2026
Merged

test(e2e): skip docker gpu test in rust suite#1103
pimlock merged 1 commit into
mainfrom
test/skip-docker-gpu-rust-e2e

Conversation

@pimlock
Copy link
Copy Markdown
Collaborator

@pimlock pimlock commented May 1, 2026

Summary

Skip the Docker GPU smoke test when running the regular Rust e2e suite. We will need to add a dedicated test for this in our E2E-GPU workflow.

Fixes failing E2E workflows on main: https://github.com/NVIDIA/OpenShell/actions/runs/25196530420/job/73879135082

Related Issue

N/A

Changes

  • Add docker_gpu_sandbox_runs_nvidia_smi to the regular e2e:rust Cargo skip list.
  • Update the comment for dedicated e2e tests to include Docker GPU coverage.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Additional testing:

  • cargo test --manifest-path e2e/rust/Cargo.toml --features e2e --no-run
  • cargo test --manifest-path e2e/rust/Cargo.toml --features e2e --test docker_gpu --no-run
  • cargo test --manifest-path e2e/rust/Cargo.toml --features e2e --test docker_gpu -- --skip docker_gpu_sandbox_runs_nvidia_smi

Note: mise run pre-commit was attempted. Rust tests passed, but license:check failed on the pre-existing untracked file test-policy.yaml missing SPDX headers; this PR does not touch that file.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
@pimlock pimlock requested a review from a team as a code owner May 1, 2026 01:44
@pimlock pimlock self-assigned this May 1, 2026
@pimlock pimlock merged commit f46b296 into main May 1, 2026
42 checks passed
@pimlock pimlock deleted the test/skip-docker-gpu-rust-e2e branch May 1, 2026 01:59
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.

2 participants