Skip to content

fix(e2e): reserve distinct corporate proxy fixture ports - #3761

Merged
elezar merged 1 commit into
mainfrom
codex/fixture-distinct-ports
Sep 28, 2026
Merged

elezar merged 1 commit into
mainfrom
codex/fixture-distinct-ports

Conversation

@drew

@drew drew commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

Reserve all four corporate proxy fixture ports simultaneously so allowed and denied endpoints cannot receive the same port. This removes a false security assertion failure observed in main-branch VM E2E.

Related Issue

No issue required: localized E2E fixture bug. See failing Release Dev run.

Changes

  • Allocate four loopback listeners before reading their assigned ports.
  • Keep the listeners alive until every port is selected, then release them for the fixture processes.

Testing

  • mise run pre-commit passes
  • mise run test passes
  • VM E2E rerun (requires release E2E environment)
  • mise run ci passes locally: Go gateway tests see an existing system gateway and fail their empty-directory assertions; unrelated to this Rust fixture change.

Checklist

  • Follows Conventional Commits
  • Commit is signed off (DCO)
  • Architecture docs not applicable: test fixture only

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@elezar elezar added the test:e2e Requires end-to-end coverage label Sep 28, 2026
@github-actions

Copy link
Copy Markdown

Label test:e2e applied for 43feeec. Open the existing run and click Re-run all jobs to execute with the label set. The run will execute the standard E2E suite after building the required gateway, sandbox, and supervisor images once. The matching required CI gate status on this PR will flip green automatically once the run finishes.

@elezar
elezar added this pull request to the merge queue Sep 28, 2026
Merged via the queue into main with commit 6f00d5c Sep 28, 2026
125 of 126 checks passed
@elezar
elezar deleted the codex/fixture-distinct-ports branch September 28, 2026 07:20
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.

3 participants