Skip to content

test(e2e): run host alias checks from docker#314

Merged
pimlock merged 2 commits intomainfrom
test/host-gateway-docker-server
Mar 15, 2026
Merged

test(e2e): run host alias checks from docker#314
pimlock merged 2 commits intomainfrom
test/host-gateway-docker-server

Conversation

@pimlock
Copy link
Collaborator

@pimlock pimlock commented Mar 15, 2026

Summary

  • run the host alias e2e helper server in a Docker container published on the host instead of a local TcpListener
  • add curl timeouts so host alias and inference-local failures stop quickly instead of hanging
  • keep the direct host alias and inference.local coverage in the same e2e test file

Related Issue

  • None

Changes

  • update e2e/rust/tests/host_gateway_alias.rs to launch a Docker-backed HTTP server and clean it up automatically
  • publish the helper server on a host port so host.openshell.internal resolves to something reachable in GitHub Actions job containers
  • add --max-time 15 to sandbox curl invocations

Testing

  • cargo test --manifest-path e2e/rust/Cargo.toml --features e2e --test host_gateway_alias

Checklist

  • Tests added or updated
  • Relevant tests passed
  • No unrelated files included

@pimlock pimlock added the test:e2e Requires end-to-end coverage label Mar 15, 2026
@pimlock pimlock merged commit 9d6d149 into main Mar 15, 2026
9 checks passed
@pimlock pimlock deleted the test/host-gateway-docker-server branch March 15, 2026 03:10
drew pushed a commit that referenced this pull request Mar 16, 2026
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.

1 participant