Skip to content

Make proxy decider test DNS independent#22540

Open
starr-openai wants to merge 1 commit into
starr/remote-ci-routing-test-harness-20260511from
starr/remote-ci-devbox-proxy-timeout-20260513
Open

Make proxy decider test DNS independent#22540
starr-openai wants to merge 1 commit into
starr/remote-ci-routing-test-harness-20260511from
starr/remote-ci-devbox-proxy-timeout-20260513

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

Why

The managed network proxy decider test is meant to prove that the decider still runs after the session starts in full-access mode and then returns to workspace-write restrictions. On Mike's devbox, the test's example.com request can be classified by ambient DNS as local/private before the decider is consulted, so the same shard becomes flaky even though the proxy behavior is correct.

What changed

  • use public IP literal 8.8.8.8 in the proxy request instead of example.com
  • keep the assertion focused on the decider path, without depending on ambient DNS resolution

Stack

  • PR #22199 Isolate tmp-dependent tests from ambient git
  • PR #22201 Use remote fs for turn diff repo root
  • PR #22200 Emit unified exec end on startup failure
  • PR #22389 Stabilize remote routing e2e tests
  • this PR Make proxy decider test DNS independent

Validation

  • On dev: bazel test //codex-rs/core:core-unit-tests --test_filter=managed_network_proxy_decider_survives_full_access_start --runs_per_test=10 --test_output=errors passed with 80 shard/run executions.
  • Broader full remote validation is being rerun from the new top of stack.

@starr-openai starr-openai requested a review from a team as a code owner May 13, 2026 19:15
Co-authored-by: Codex <noreply@openai.com>
@starr-openai starr-openai force-pushed the starr/remote-ci-devbox-proxy-timeout-20260513 branch from 62c8bb9 to 4ea1f18 Compare May 13, 2026 19:17
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.

1 participant