fix: bump pinned gVisor release from 20250623.0 to 20250707.0#45101
Conversation
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the pinned gVisor runtime release to restore generated installation steps.
Changes:
- Bumps
DefaultGVisorVersionto20250707.0. - Synchronizes the constants documentation.
Show a summary per file
| File | Description |
|---|---|
pkg/constants/version_constants.go |
Updates the pinned gVisor release. |
pkg/constants/README.md |
Documents the updated release. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Medium
|
✅ Design Decision Gate 🏗️ completed the design decision gate check. No ADR enforcement needed: PR #45101 does not have the 'implementation' label and has ≤100 new lines of code in business logic directories (default_business_additions=2). |
|
✅ Test Quality Sentinel completed test quality analysis. No test files were added or modified in this PR. This PR only bumps the gVisor release version constant (pkg/constants/version_constants.go). Test Quality Sentinel skipped. |
|
✅ PR Code Quality Reviewer completed the code quality review. |
|
🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅ |
There was a problem hiding this comment.
Clean version bump. The constant and its README documentation comment are updated consistently. No issues found.
🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · 11.1 AIC · ⌖ 7.54 AIC · ⊞ 4.8K
There was a problem hiding this comment.
Skills-Based Review 🧠
Applied /diagnosing-bugs — this is a targeted hotfix with a clear root cause (404 on the deleted release artifact) and a minimal, correct fix.
The change is surgical: one constant updated in version_constants.go, documentation kept in sync in README.md. No broader logic is affected. ✅
🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · 12.5 AIC · ⌖ 4.29 AIC · ⊞ 6.6K
Comment /matt to run again
There was a problem hiding this comment.
✅ LGTM
Minimal, correct version bump. DefaultGVisorVersion and its README doc comment are updated in sync from 20250623.0 → 20250707.0. No logic changes, no test coverage gaps, no other concerns.
🔎 Code quality review by PR Code Quality Reviewer · 18.8 AIC · ⌖ 4.28 AIC · ⊞ 5.4K
Comment /review to run again
|
🎉 This pull request is included in a new release. Release: |
Release
20250623.0no longer exists onstorage.googleapis.com/gvisor/releases/, causing all workflows withsandbox.agent.runtime: gvisorto fail at the "Install gVisor (runsc)" step with a 404.Changes
pkg/constants/version_constants.go— bumpsDefaultGVisorVersionfrom20250623.0→20250707.0pkg/constants/README.md— updates the inline documentation comment to match