Share benchmark execution and preserve reviewed evidence - #25
Merged
Merged
Conversation
# Conflicts: # data/latest/series.json # migration/evidence.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Benchmark cases now select reviewed workloads through shared execution, provider setup, timing, output verification, and reporting. The common Docker action controls cache publication for both providers: cold publishes, ordinary warm replay restores, and explicitly declared warm publication applies to both arms. Case actions retain upstream preparation, recipe inputs, and correctness checks. Roughly 600 lines of repeated Docker lifecycle code are removed.
PostHog's missing native warm scope and inspection inside the build timer are fixed. The common scope helper preserves series continuity and legacy cold-scope handoff. Hugo's loaded-image projection disables incompatible attestations in both providers. Regressions execute scope selection and the phase-policy CLI, verify every image inspection is outside timing, and check that the product contract sees the correct adapters and literal trust policies.
The sole One release pin remains in the provider wrapper. A small JavaScript GitHub post hook uploads the original product evidence after One's cleanup, using the official artifact SDK; Ruby owns the harness and reports. Missing evidence fails retention. Dependencies install outside timing and are excluded from case digests and prepared-source copies; their lockfile is included. The existing product pin-sync path still finds the wrapper pin.
bin/bench collectimports original phase artifacts from a verified export matched to its dispatch receipt, rejects conflicting observations, checks workflow/job/post-step completion, and generates the report. It can resume an identical import. Reports and the catalog retain failed, incomplete, unfavorable, and methodologically invalid observations. Documentation describes the complete case, series, collection, and review process.Six previously completed Docker requests are imported and independently archived: PostHog layers/combined, Hugo, Mastodon streaming, and n8n runners/distroless. Plan-bound methodology reviews now mark all affected earlier Docker series invalid for comparative claims because Actions Cache exported warm state while BoringCache restored without publishing. Their original measurements remain unchanged. The six published bundles were downloaded again, checked against SHA-256, extracted, and verified against 78 listed files with no scoped gaps. Older output failures, cancelled requests, Deno native errors, and unmeasured storage remain visible.
Canary monitoring follows each active historical schedule until its explicit cutover. A read-only collection verified 16 dispatch receipts and 17 child runs on October 4. Missing, stale, or failed canaries still fail monitoring. Imported case definitions do not imply an active central schedule.
Validation: Ruby tests, Node evidence-retention regression, 68 case definitions, 22 publication entries, 62 report workflow contracts, 70 execution views, product cache-interface checks, and Actionlint pass. Hugo Go and corrected Hugo Docker completed cold/warm execution and retained final product evidence after cleanup. Five completed requests, including three failures before build, were independently archived and verified against 53 listed files. Their interpretations retain slower BoringCache warm results and unmeasured Actions Cache storage. Initial Docker lifecycle requests exposed an output-name mismatch before build; those failures are preserved. Separate signed correction plans rerun Hugo and both PostHog variants. Both corrected PostHog variants now pass cold/warm output and completion checks. Their added archives were independently downloaded and verified against 30 listed files, bringing the shared-lifecycle release to seven exports and 83 verified files. The selected Docker-tag storage in the combined profile is not presented as total tool-cache storage; identical-source replay does not isolate tool-cache benefit.
Remaining consolidation gates are explicit: qualify the other active variants and rolling seed lineage, switch schedules/source-sync/release callers, resolve shared GitHub cache capacity, verify the central reporting feed and reviewed website claims, and audit selected fork evidence before retirement. Original repositories remain active. This PR does not establish complete migration, publication approval, or permission to delete repositories.