test(e2e): pin verifier v0.8.1 and proxy v0.5.1 release cuts before tagging - #29
Merged
Merged
Conversation
…agging Both patches carry only the shutdown reason-consistency fix and a doc correction (verifier #212 / proxy #83), cut in #213 and #84 (+ #85 wording). Pinned to the exact commits that will be tagged, so the run answers whether those trees work together before the tags exist — the order the release rule requires, and the opposite of how v0.8.0 / v0.5.0 went out. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
The change is a straightforward update of pinned component SHAs consistent with the stated pre-tag E2E workflow and does not introduce functional/test logic changes.
Pull request overview
Pins the E2E harness in this repo to the specific merged commit SHAs intended for the upcoming patch releases, so the umbrella cross-component E2E run is green before tagging proxy/verifier releases.
Changes:
- Update
PROXY_REVto9fc0dab6…(targeting proxyv0.5.1). - Update
VERIFIER_REVtodd8adc65…(targeting verifierv0.8.1).
File summaries
| File | Description |
|---|---|
| Makefile | Updates pinned component revisions used by make setup/build/test-e2e to run pre-tag E2E against the intended patch-release commits. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Pre-tag E2E for two patch releases, in the order the release rule requires: cut merged → umbrella E2E green → tag.
c5e0608c(v0.8.0)dd8adc65(#213)v0.8.19828a137(v0.5.0)9fc0dab6(#84 + #85)v0.5.1Both patches contain only the standalone-template / proxy shutdown fix from review —
reasonon the finalgraceful shutdown: completeline now names what set the exit code, with the drain outcome under its owndrainkey — plus theexitoption doc pointing atdeferExit. Fixed-only, no BREAKING.PROVIDER_REV(v0.12.0) andPROVIN_REVare unchanged. Tags are pushed only after this run is green.🤖 Generated with Claude Code