Update golang version to 1.26.3#461
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
📝 WalkthroughWalkthroughThis PR upgrades the Go toolchain from 1.25.x to 1.26.3 across the module declaration, default and FIPS builder images (pinned digests for FIPS), Makefile FIPS variables, and the e2e test container image tag. ChangesGo 1.26 Upgrade
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Makefile`:
- Line 18: Replace the mutable tag assigned to BUILD_IMAGE_FIPS with the exact
patch+digest used in Dockerfile.fips so both build paths use the identical image
reference; locate the BUILD_IMAGE_FIPS variable in the Makefile and update its
value from cgr.dev/mattermost.com/go-msft-fips:1.26 to the pinned reference
matching Dockerfile.fips
(1.26.3@sha256:90017f556002f392c7eb21a96bf845e1b0675cb32a43ac8c85c25cbb80e848cc).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 8f1391f2-13aa-4f61-83c3-0343fbd9a0ea
📒 Files selected for processing (5)
DockerfileDockerfile.fipsMakefilego.modtest/e2e.sh
Align the Makefile FIPS build image reference with the pinned digest used in Dockerfile.fips so both build paths use the identical image. Co-authored-by: Cursor <cursoragent@cursor.com>
Pin the FIPS base image to v16.1.0 with its digest in both Dockerfile.fips and Makefile so direct docker builds and Makefile-driven builds use the same image. Drop the now-obsolete TODO since the digest is pinned. Co-authored-by: Cursor <cursoragent@cursor.com>
|
@esarafianou: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. DetailsI understand the commands that are listed here |
Summary
Bumps Go from 1.25.9 to 1.26.3 across the operator and updates FIPS image references:
1.26.3cgr.dev/mattermost.com/glibc-openssl-fipsbumped to16.1.0@sha256:3ffd0b8b...Release Note
Made with Cursor