Skip to content

chore: Upgrade golang version to 1.26.2#10043

Merged
alphanota merged 6 commits intoGoogleContainerTools:mainfrom
menahyouyeah:go-upgrade
Apr 8, 2026
Merged

chore: Upgrade golang version to 1.26.2#10043
alphanota merged 6 commits intoGoogleContainerTools:mainfrom
menahyouyeah:go-upgrade

Conversation

@menahyouyeah
Copy link
Copy Markdown
Contributor

Upgrade golang version is skaffold repo to 1.26.2

@menahyouyeah menahyouyeah requested a review from a team as a code owner April 8, 2026 16:11
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Go version from 1.25.7 to 1.26.2 in both the Dockerfile dependencies and the go.mod file. A critical issue was identified where the go.sum and vendor/ directory were not updated to reflect this change, which will lead to build failures in CI due to the project's reliance on vendored modules.

Comment thread go.mod
module github.com/GoogleContainerTools/skaffold/v2

go 1.25.8
go 1.26.2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The Go version has been updated to 1.26.2, but the pull request does not include the corresponding updates to the go.sum file and the vendor/ directory (specifically vendor/modules.txt). Since the project's Makefile (lines 80 and 101) uses the -mod=vendor flag for builds, the build will fail in CI environments that enforce vendor consistency because the Go version in go.mod will not match the one recorded in vendor/modules.txt. Please run go mod tidy and go mod vendor and include the resulting changes in this pull request.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix.

@menahyouyeah menahyouyeah added the kokoro:force-run forces a kokoro re-run on a PR label Apr 8, 2026
@kokoro-team kokoro-team removed the kokoro:force-run forces a kokoro re-run on a PR label Apr 8, 2026
@menahyouyeah menahyouyeah added the kokoro:force-run forces a kokoro re-run on a PR label Apr 8, 2026
@kokoro-team kokoro-team removed the kokoro:force-run forces a kokoro re-run on a PR label Apr 8, 2026
@alphanota alphanota merged commit c186fff into GoogleContainerTools:main Apr 8, 2026
13 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants