Skip to content

[codex] Restore remote exec-server filesystem tests#17989

Merged
starr-openai merged 1 commit intomainfrom
starr/restore-remote-fs-tests
Apr 15, 2026
Merged

[codex] Restore remote exec-server filesystem tests#17989
starr-openai merged 1 commit intomainfrom
starr/restore-remote-fs-tests

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

Summary

  • Re-enable remote variants for the exec-server filesystem sandbox/symlink tests that were made local-only in PR Stabilize exec-server filesystem tests in CI #17671.
  • Restore use_remote parameterization for the readable-root, normalized symlink escape, symlink removal, and symlink copy-preservation cases.
  • Preserve mode={use_remote} context on key async filesystem failures so CI failures point at the local or remote lane.

Validation

  • cd codex-rs && just fmt
  • Not run: bazel test //codex-rs/exec-server:exec-server-file_system-test per local Codex development guidance to avoid test runs unless explicitly requested.

Re-enable remote coverage for the sandbox and symlink filesystem cases that were made local-only while stabilizing the remote harness.

Co-authored-by: Codex <noreply@openai.com>
@starr-openai starr-openai marked this pull request as ready for review April 15, 2026 20:21
@starr-openai starr-openai deleted the starr/restore-remote-fs-tests branch April 15, 2026 20:28
@starr-openai starr-openai restored the starr/restore-remote-fs-tests branch April 15, 2026 20:43
@starr-openai starr-openai reopened this Apr 15, 2026
@starr-openai starr-openai enabled auto-merge (squash) April 15, 2026 20:43
@starr-openai starr-openai merged commit ba36415 into main Apr 15, 2026
96 of 100 checks passed
@starr-openai starr-openai deleted the starr/restore-remote-fs-tests branch April 15, 2026 20:48
@github-actions github-actions bot locked and limited conversation to collaborators Apr 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants