Skip to content

Add bounded HQStore backstop reclaim assertions - #5

Open
quad341 wants to merge 1 commit into
builder/ga-7a0x3.6from
builder/ga-7a0x3.7
Open

quad341 wants to merge 1 commit into
builder/ga-7a0x3.6from
builder/ga-7a0x3.7

Conversation

@quad341

@quad341 quad341 commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a coordstore benchmark backstop assertion contract with per-tier reclaim counts
  • expose detailed HQStore backstop reclaim counts through the benchmark adapter
  • run the HQStore backstop assertion after the coordstore workload warm-up/run

Verification

  • go test ./internal/benchmarks/coordstore -run 'TestCheckBackstopReclaim|TestBackstopCheckFailureReason' -count=1\n- go test ./internal/benchmarks/coordstore -run 'TestBenchmarkSuite/hqstore' -count=1 -v\n- go test ./internal/benchmarks/coordstore/... -count=1\n- go test ./internal/beads -run 'TestHQStore' -count=1\n- go test ./internal/testenv -run TestRequiresDedicatedTestenvImportFile -count=1\n- go vet ./...\n\n## Known Baseline\n- make test-fast-parallel fails on nested worktrees/ga-gjnt7 scanner/doc coverage: TestNoBdExecOutsideBeads, TestWalkthroughURLStringsStayInContractFile, TestDocDirCoverage. Final manual log: /tmp/gc-local-tests.M2xryF; pre-commit log: /tmp/gc-local-tests.Z5jiRW.\n- One fast-target run also hit load-sensitive TestExecCommandRunnerStopsBDSlowTimerForFastBDCommand; focused rerun passed.\n\nBead: ga-7a0x3.7

quad341 pushed a commit that referenced this pull request Oct 11, 2026
…stale apt index) (gastownhall#7554)

## Summary

**This fixes the fork pool outage.** Since about 20:05Z on 2026-10-10,
every fork pool VM has failed setup with `apt-get install` exit 100. The
image's apt index names `libxml2 2.9.14+dfsg-1.3ubuntu3.9`, which the
Blacksmith mirror now 404s, so fork lanes have had no workers (114
actions queued).

This bumps `gastownhall/rbe-worker` from `421eb59c` to `3bfd8aea`:
- #6 refreshes the apt index once and retries the toolset install,
before the worker-env measurement (which measures upstream major.minor
only).
- #5 (S4b) and its revert #7 cancel out, so S4b doesn't ship in this
bump.

## Test plan
- [x] rbe-worker `product-check` at `3bfd8aea` against this tree: ok
- [x] `go test ./scripts/ -run 'RBE|Worker|Pool'`
- [ ] After merge: fork pool runs succeed, and fork queued drains on
core

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant