Skip to content

test(compile): allow a 1.2x wall-time cache-write overhead in the fresh-process guard - #1063

Closed
kvlonge wants to merge 1 commit into
mainfrom
test/cache-write-wall-limit
Closed

kvlonge wants to merge 1 commit into
mainfrom
test/cache-write-wall-limit

Conversation

@kvlonge

@kvlonge kvlonge commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Why

The fresh-process compile cache guard keeps failing on wall-time noise from shared runners: for example, a cache-write wall ratio of 1.179 against the 1.15 limit, while the CPU ratio stayed at 1.057, the same as passing runs. Each rerun costs a full CI round.

Changes

  • Raise only the cache-write wall-time overhead limit from 1.15 to 1.20.
  • Leave the CPU overhead limits (1.10 and 1.15 for large projects) unchanged. CPU time is what a real cache-write cost shows up in.

Verification

  • Constant-only change. CI runs the guard.

…sh-process guard

Shared CI runners swing wall time while CPU stays flat; the CPU overhead
limits are unchanged and still catch real cache-write cost.
@kvlonge
kvlonge enabled auto-merge October 9, 2026 09:24
@kvlonge
kvlonge added this pull request to the merge queue Oct 9, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 9, 2026
@kvlonge
kvlonge added this pull request to the merge queue Oct 9, 2026
@kvlonge
kvlonge removed this pull request from the merge queue due to a manual request Oct 9, 2026
@kvlonge

kvlonge commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

Landed together in #1075 to cut merge-queue CI runs; review history stays here.

@kvlonge kvlonge closed this Oct 9, 2026
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