fix(renovate): move the Talos machinery pin with Talos and list the manual pins on the PR - #4043
Conversation
…anual pins on the PR Groups github.com/siderolabs/talos/pkg/machinery with siderolabs/talos so both move in one PR, and adds prBodyNotes naming the pins Renovate cannot derive (TALOSCTL_SHA256, the Hetzner iso id, kubernetesVersion). Fixes #3734 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…pins scripts/tests/test-talos-apparmor-lsm.sh pins the Talos version it was last checked against and fails on any bump until someone re-reads that release's kernel CONFIG_LSM (seen on #3713 once its digest pin was fixed). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai review |
Readiness record @de68c389:
|
|
✅ Action performedReview finished.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📜 Recent review details🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe Talos Renovate rule now groups Priority: ➖ Normal Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to This change groups related Talos updates and documents the manual pins required for a complete upgrade. No material merge risk remains. Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (4 passed)
Full details: Linked Issues checkExplanation The change addresses the coding objectives in Resolution Verify the next real Talos bump. Confirm that the grouped machinery update is included and that 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. Comment |
Why
Every Talos upgrade arrives with failing checks because two related pins do not move with it. Each time, someone has to rediscover that by hand, one CI cycle per missing pin.
What
The Talos library the compatibility check reads now updates in the same pull request as Talos itself. Each Talos pull request also carries a checklist of the pins that cannot be updated automatically, including where to find the checksum value. Proof comes with the next Talos release, so the issue moves to Verifying after merge.
Fixes #3734