Skip to content

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

Merged
drew merged 10 commits into
mainfrom
codex-docker-gateway-e2e
May 5, 2026
Merged

test(e2e): run suites against docker gateway#1153
drew merged 10 commits into
mainfrom
codex-docker-gateway-e2e

Conversation

@drew

@drew drew commented May 4, 2026

Copy link
Copy Markdown
Collaborator

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

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown

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-pr-bot Bot commented May 4, 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.

@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-pr-bot Bot commented May 4, 2026

Copy link
Copy Markdown

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

drew commented May 4, 2026

Copy link
Copy Markdown
Collaborator Author

/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

drew commented May 4, 2026

Copy link
Copy Markdown
Collaborator Author

/ok to test b3ceeed

@drew

drew commented May 5, 2026

Copy link
Copy Markdown
Collaborator Author

/ok to test c402b01

@drew

drew commented May 5, 2026

Copy link
Copy Markdown
Collaborator Author

/ok to test 0913272

@drew

drew commented May 5, 2026

Copy link
Copy Markdown
Collaborator Author

/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
@drew
drew marked this pull request as ready for review May 5, 2026 19:40
@drew
drew merged commit e4b4e92 into main May 5, 2026
29 of 32 checks passed
@drew
drew deleted the codex-docker-gateway-e2e branch May 5, 2026 20: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