Skip to content

ci: stabilize Windows test-cost ratchet - #587

Merged
matdev83 merged 3 commits into
mainfrom
fix/windows-test-cost-stability
Sep 3, 2026
Merged

ci: stabilize Windows test-cost ratchet#587
matdev83 merged 3 commits into
mainfrom
fix/windows-test-cost-stability

Conversation

@matdev83

@matdev83 matdev83 commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Changes

  • allow the observed two-process variance in the short quality-check measurement
  • give the complete three-target Windows ratchet 30 minutes instead of timing out during the final head measurement

Evidence

  • repeated main run completed test-unit comparison successfully
  • quality-checks used 29 processes vs an allowed 27; this raises only the additive tolerance from 6 to 8
  • the same run reached qa-tagged-hotspots head measurement at the former 20-minute deadline
  • go test -count=1 ./tools/testcost ./tools/taskrunner
  • authorized ratchet preflight passed with LIP_ALLOW_TEST_COST_GROWTH=1

@matdev83 matdev83 added the allow-test-cost-growth Authorize an intentional test-cost policy update without bypassing measurement label Sep 3, 2026
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: 76fca77f-9bba-4101-9e26-50faf1455b2a


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@matdev83
matdev83 merged commit a2845f6 into main Sep 3, 2026
24 checks passed
@matdev83
matdev83 deleted the fix/windows-test-cost-stability branch September 3, 2026 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow-test-cost-growth Authorize an intentional test-cost policy update without bypassing measurement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant