Skip to content

fix(ci): --no-fail-fast, deflake remaining sleeper copy, regression test - #733

Merged
danshapiro merged 1 commit into
mainfrom
the-usual/ci-rust-test-followup
Sep 7, 2026
Merged

fix(ci): --no-fail-fast, deflake remaining sleeper copy, regression test#733
danshapiro merged 1 commit into
mainfrom
the-usual/ci-rust-test-followup

Conversation

@danshapiro

Copy link
Copy Markdown
Owner

Follow-up to PR #724 addressing delta-review findings:

  1. --no-fail-fast on workspace tests so one flake doesn't hide all later test binaries
  2. Deflake restore_spawn_gate.rs — same ETXTBSY race as common/mod.rs, now fixed with AtomicU64 counter
  3. Regression test for shared common::sleeper_cli_spec uniqueness (matching 1839b11 precedent)

Kata 299r tracks the auto_resume_e2e timing flakes that block rust-test from going green on 4-core CI.

… add regression test

Delta review of PR #724 found four actionable findings:

1. Missing --no-fail-fast: one flake aborted the entire workspace test run,
   hiding all later test binaries. Add --no-fail-fast so all binaries run.

2. restore_spawn_gate.rs had the same ETXTBSY race as common/mod.rs (shared
   {name}-{pid} path, 10 parallel tests). Add AtomicU64 counter matching
   the common/mod.rs fix.

3. No regression test for the shared common::sleeper_cli_spec uniqueness.
   Add shared_sleeper_cli_spec_paths_are_unique_per_call in pane_ledger_triggers
   matching the existing test in cross_kind_liveness.rs (commit 1839b11).

Kata 299r tracks the auto_resume_e2e timing flakes that block rust-test
from going green on 4-core CI runners.
@danshapiro
danshapiro merged commit de16620 into main Sep 7, 2026
3 of 4 checks passed
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