Skip to content

test(e2e): run suites against docker gateway#1153

Draft
drew wants to merge 9 commits intomainfrom
codex-docker-gateway-e2e
Draft

test(e2e): run suites against docker gateway#1153
drew wants to merge 9 commits intomainfrom
codex-docker-gateway-e2e

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented May 4, 2026

Summary

Switch normal non-GPU e2e runs to a standalone Docker compute-driver gateway, while allowing callers to reuse an existing plaintext gateway through OPENSHELL_GATEWAY_ENDPOINT.

Related Issue

N/A

Changes

  • Added a shared e2e/with-docker-gateway.sh wrapper for ephemeral Docker-gateway setup or existing HTTP endpoint reuse.
  • Updated mise run e2e, e2e:rust, and e2e:python to avoid the k3s cluster task.
  • Removed the gateway_resume lifecycle e2e test and CI matrix lane.
  • Updated non-GPU e2e CI to stop pulling, building, and bootstrapping cluster images.
  • Updated TESTING.md for Docker gateway defaults and OPENSHELL_GATEWAY_ENDPOINT.

Testing

  • bash -n e2e/with-docker-gateway.sh e2e/rust/e2e-docker.sh
  • OPENSHELL_GATEWAY_ENDPOINT=http://127.0.0.1:18080 e2e/with-docker-gateway.sh true
  • OPENSHELL_GATEWAY_ENDPOINT=https://127.0.0.1:18080 e2e/with-docker-gateway.sh true rejects HTTPS raw endpoint mode
  • cargo test --manifest-path e2e/rust/Cargo.toml --features e2e --no-run
  • mise run --no-deps python:proto
  • uv run pytest python/
  • mise run pre-commit
  • Full live mise run e2e:rust and mise run e2e:python were not run locally

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not changed in this PR)

@drew drew requested a review from a team as a code owner May 4, 2026 19:07
@drew drew added the test:e2e Requires end-to-end coverage label May 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Label test:e2e applied for de67bcf. Open the existing run and click Re-run all jobs to execute with the label set. The E2E Gate check on this PR will flip green automatically once the run finishes.

@drew drew marked this pull request as draft May 4, 2026 21:51
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 4, 2026

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 force-pushed the codex-docker-gateway-e2e branch from a649023 to f6f286b Compare May 4, 2026 21:58
@drew drew marked this pull request as ready for review May 4, 2026 22:01
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 4, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@drew
Copy link
Copy Markdown
Collaborator Author

drew commented May 4, 2026

/ok to test e9c0fe7

@drew drew force-pushed the codex-docker-gateway-e2e branch from e9c0fe7 to b3ceeed Compare May 4, 2026 23:44
@drew
Copy link
Copy Markdown
Collaborator Author

drew commented May 4, 2026

/ok to test b3ceeed

@drew
Copy link
Copy Markdown
Collaborator Author

drew commented May 5, 2026

/ok to test c402b01

@drew
Copy link
Copy Markdown
Collaborator Author

drew commented May 5, 2026

/ok to test 0913272

@drew
Copy link
Copy Markdown
Collaborator Author

drew commented May 5, 2026

/ok to test 134dd46

TaylorMutch
TaylorMutch previously approved these changes May 5, 2026
@drew drew force-pushed the codex-docker-gateway-e2e branch from da288d1 to 3bfa03c Compare May 5, 2026 04:39
@drew drew marked this pull request as draft May 5, 2026 06:28
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