From 595f09d6e254fba52ff7c1ec88958fb06d310711 Mon Sep 17 00:00:00 2001 From: Adam Perry Date: Mon, 22 Jun 2026 20:49:20 +0000 Subject: [PATCH] bazel: document Windows test tag exceptions (#28598) --- codex-rs/agent-identity/BUILD.bazel | 3 +++ codex-rs/apply-patch/BUILD.bazel | 3 +++ codex-rs/file-system/BUILD.bazel | 3 +++ codex-rs/memories/read/BUILD.bazel | 3 +++ codex-rs/secrets/BUILD.bazel | 3 +++ codex-rs/state/BUILD.bazel | 3 +++ codex-rs/utils/approval-presets/BUILD.bazel | 3 +++ codex-rs/utils/rustls-provider/BUILD.bazel | 3 +++ codex-rs/windows-sandbox-rs/BUILD.bazel | 3 +++ 9 files changed, 27 insertions(+) diff --git a/codex-rs/agent-identity/BUILD.bazel b/codex-rs/agent-identity/BUILD.bazel index 5f49ffb72e..1e5aecd85e 100644 --- a/codex-rs/agent-identity/BUILD.bazel +++ b/codex-rs/agent-identity/BUILD.bazel @@ -3,6 +3,9 @@ load("//:defs.bzl", "codex_rust_crate") codex_rust_crate( name = "agent-identity", crate_name = "codex_agent_identity", + # This test is short once running, but it hit the 60s Windows timeout when + # several freshly invalidated tests launched together. Serialize local runs + # to avoid runner startup contention while keeping the small test size. test_tags_by_target = { "agent-identity-unit-tests-windows-cross": ["exclusive-if-local"], }, diff --git a/codex-rs/apply-patch/BUILD.bazel b/codex-rs/apply-patch/BUILD.bazel index 80907450d2..081c3ffd30 100644 --- a/codex-rs/apply-patch/BUILD.bazel +++ b/codex-rs/apply-patch/BUILD.bazel @@ -6,6 +6,9 @@ codex_rust_crate( test_sizes = { "apply-patch-all-test-windows-cross": "medium", }, + # Cold Windows runs take about 54s, while remote results have reported + # about 1s and made the medium budget look oversized. Run it alone and + # bypass remote test-result caching so CI measures the cold runtime. test_tags_by_target = { "apply-patch-all-test-windows-cross": [ "exclusive-if-local", diff --git a/codex-rs/file-system/BUILD.bazel b/codex-rs/file-system/BUILD.bazel index 4fe0ec7a9d..1a851d07e2 100644 --- a/codex-rs/file-system/BUILD.bazel +++ b/codex-rs/file-system/BUILD.bazel @@ -3,6 +3,9 @@ load("//:defs.bzl", "codex_rust_crate") codex_rust_crate( name = "file-system", crate_name = "codex_file_system", + # This test is short once running, but it hit the 60s Windows timeout when + # several freshly invalidated tests launched together. Serialize local runs + # to avoid runner startup contention while keeping the small test size. test_tags_by_target = { "file-system-unit-tests-windows-cross": ["exclusive-if-local"], }, diff --git a/codex-rs/memories/read/BUILD.bazel b/codex-rs/memories/read/BUILD.bazel index fb101c8513..ea0571c2d5 100644 --- a/codex-rs/memories/read/BUILD.bazel +++ b/codex-rs/memories/read/BUILD.bazel @@ -3,6 +3,9 @@ load("//:defs.bzl", "codex_rust_crate") codex_rust_crate( name = "read", crate_name = "codex_memories_read", + # This test is short once running, but it hit the 60s Windows timeout when + # several freshly invalidated tests launched together. Serialize local runs + # to avoid runner startup contention while keeping the small test size. test_tags_by_target = { "read-unit-tests-windows-cross": ["exclusive-if-local"], }, diff --git a/codex-rs/secrets/BUILD.bazel b/codex-rs/secrets/BUILD.bazel index 0fefc47377..a9c47f8963 100644 --- a/codex-rs/secrets/BUILD.bazel +++ b/codex-rs/secrets/BUILD.bazel @@ -3,6 +3,9 @@ load("//:defs.bzl", "codex_rust_crate") codex_rust_crate( name = "secrets", crate_name = "codex_secrets", + # This test is short once running, but it hit the 60s Windows timeout when + # several freshly invalidated tests launched together. Serialize local runs + # to avoid runner startup contention while keeping the small test size. test_tags_by_target = { "secrets-unit-tests-windows-cross": ["exclusive-if-local"], }, diff --git a/codex-rs/state/BUILD.bazel b/codex-rs/state/BUILD.bazel index fc97b48e14..595ebef3a5 100644 --- a/codex-rs/state/BUILD.bazel +++ b/codex-rs/state/BUILD.bazel @@ -12,6 +12,9 @@ codex_rust_crate( test_sizes = { "state-unit-tests-windows-cross": "medium", }, + # Cold Windows runs exceed the 60s short timeout, while remote results have + # reported about 15s and made the medium budget look oversized. Run it alone + # and bypass remote test-result caching so CI measures the cold runtime. test_tags_by_target = { "state-unit-tests-windows-cross": [ "exclusive-if-local", diff --git a/codex-rs/utils/approval-presets/BUILD.bazel b/codex-rs/utils/approval-presets/BUILD.bazel index c66b9887b5..c8b68305c2 100644 --- a/codex-rs/utils/approval-presets/BUILD.bazel +++ b/codex-rs/utils/approval-presets/BUILD.bazel @@ -3,6 +3,9 @@ load("//:defs.bzl", "codex_rust_crate") codex_rust_crate( name = "approval-presets", crate_name = "codex_utils_approval_presets", + # This test is short once running, but it hit the 60s Windows timeout when + # several freshly invalidated tests launched together. Serialize local runs + # to avoid runner startup contention while keeping the small test size. test_tags_by_target = { "approval-presets-unit-tests-windows-cross": ["exclusive-if-local"], }, diff --git a/codex-rs/utils/rustls-provider/BUILD.bazel b/codex-rs/utils/rustls-provider/BUILD.bazel index 65c6206104..7d85da4261 100644 --- a/codex-rs/utils/rustls-provider/BUILD.bazel +++ b/codex-rs/utils/rustls-provider/BUILD.bazel @@ -6,6 +6,9 @@ codex_rust_crate( test_sizes = { "rustls-provider-preinstalled-test-windows-cross": "medium", }, + # This unit test is short once running, but it hit the 60s Windows timeout + # when several freshly invalidated tests launched together. Serialize local + # runs to avoid runner startup contention while keeping its small size. test_tags_by_target = { "rustls-provider-unit-tests-windows-cross": ["exclusive-if-local"], }, diff --git a/codex-rs/windows-sandbox-rs/BUILD.bazel b/codex-rs/windows-sandbox-rs/BUILD.bazel index 92516ae4e7..d05dee7900 100644 --- a/codex-rs/windows-sandbox-rs/BUILD.bazel +++ b/codex-rs/windows-sandbox-rs/BUILD.bazel @@ -9,6 +9,9 @@ codex_rust_crate( test_sizes = { "windows-sandbox-rs-unit-tests-windows-cross": "medium", }, + # Cold Windows runs exceed the 60s short timeout, while remote results have + # reported about 3s and made the medium budget look oversized. Run it alone + # and bypass remote test-result caching so CI measures the cold runtime. test_tags_by_target = { "windows-sandbox-rs-unit-tests-windows-cross": [ "exclusive-if-local",