Skip to content

fix(pvisor): separate network-only proxying from Gateway routing - #130

Merged
reiase merged 3 commits into
mainfrom
feat/pvisor-overlaynet-gateway-isolation
Sep 10, 2026
Merged

fix(pvisor): separate network-only proxying from Gateway routing#130
reiase merged 3 commits into
mainfrom
feat/pvisor-overlaynet-gateway-isolation

Conversation

@lizhicui

Copy link
Copy Markdown
Collaborator

Summary

  • keep allow-listed absolute-URI requests such as /v1/responses on the transparent proxy path when Gateway is off, preventing network-only pVisor runs from failing with no route for model; reject relative Gateway requests with HTTP 400 in this mode
  • stop injecting LLM SDK base URLs in network-only runs while retaining HTTP proxy configuration, existing CLI flags, and enabled/standalone Gateway routing behavior
  • add two regressions covering network-only forwarding without model routes and enabled Gateway model forwarding/rewriting

Validation

  • RUSTC_WRAPPER= cargo nextest run --locked -p persisting-gateway --test network_policy_http --filter-expr 'test(e2e_absolute_uri_llm_keeps_gateway_model_routing) | test(e2e_network_only_allowlist_forwards_absolute_llm_uri)' — both regressions passed
  • RUSTC_WRAPPER= just test persisting-overlaynet — 66 passed
  • RUSTC_WRAPPER= just test persisting-agentctl — 27 passed
  • Linux amd64 release binary built with cargo zigbuild; the build required a temporary fuser build-script host/target detection workaround, restored after compilation and excluded from this PR
  • Debian bookworm slim container: version, Run help, and a basic network-enabled Run smoke check succeeded; namespace isolation was unavailable and reported a best-effort fallback
  • pVisor package tests remain blocked on Intel macOS by libkrun's kvm_bindings dependency; container HTTP allow/deny end-to-end verification is incomplete

@reiase
reiase merged commit d3053c5 into main Sep 10, 2026
24 of 27 checks passed
@reiase
reiase deleted the feat/pvisor-overlaynet-gateway-isolation branch September 10, 2026 12:12
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