chore: update go version used in the published container#9642
chore: update go version used in the published container#9642plumpy merged 1 commit intoGoogleContainerTools:mainfrom
Conversation
|
Gemini encountered an error creating the summary. You can try again by commenting |
ghost
left a comment
There was a problem hiding this comment.
Hi team,
I'm currently reviewing this pull request and will provide my feedback shortly. In the meantime, here's a quick summary for you and other reviewers to get up to speed:
This pull request, authored by plumpy, addresses a failing post-submit build caused by an incompatibility between the Go version used in the published container (1.23.3) and a dependency (github.com/jedisct1/go-minisign) requiring Go 1.23.4 or higher.
Intent: To fix the failing post-submit build by updating the Go version used in the published container image.
Changes:
- The primary change is in the
deploy/skaffold/Dockerfile.depsfile. The Go version specified in theCOPYinstruction has been updated fromgolang:1.23.3togolang:1.23.4.
Specifically, lines 166 and 167 of deploy/skaffold/Dockerfile.deps were modified:
| Line Number | Before Change | After Change |
|---|---|---|
| 166 | COPY --from=golang:1.23.3 /usr/local/go /usr/local/go |
COPY --from=golang:1.23.4 /usr/local/go /usr/local/go |
This ensures that the container uses a compatible Go version to resolve the dependency issue.
Activity: The pull request description clearly states the problem and its solution. No other activity is currently visible.
A little haiku to brighten your day:
Go version's update,
Fixes build, smooths the workflow,
Code now compiles.
ghost
left a comment
There was a problem hiding this comment.
This change updates the Go version used in the Dockerfile to 1.23.4, which seems to be the correct fix for the build issue mentioned in the PR description. The change is small and targeted.
ghost
left a comment
There was a problem hiding this comment.
Hi team,
I'm currently reviewing this pull request and will provide my feedback shortly. In the meantime, here's a summary to help everyone get up to speed:
This pull request, authored by plumpy, addresses a post-submit build failure caused by an incompatibility between the go-minisign library and the Go version used in the published Skaffold container. The specific error message indicated a need for Go version 1.23.4 or higher, while the existing build used 1.23.3.
Intent: To resolve the post-submit build failure by upgrading the Go version used in the Skaffold container image.
Changes:
The primary change is in the deploy/skaffold/Dockerfile.deps file. Specifically, line 166 was modified to update the base Go image used for building the Skaffold container from golang:1.23.3 to golang:1.23.4.
- File Changed:
deploy/skaffold/Dockerfile.deps - Line Changed: 166
- Change: Updated the
COPYinstruction to usegolang:1.23.4instead ofgolang:1.23.3.
This simple change ensures the container uses a compatible Go version, resolving the build failure.
Here's a little haiku to brighten your day:
Go version's upgrade,
Fixes the build, smooth and swift,
Code now compiles!
Thanks,
Gemini
ghost
left a comment
There was a problem hiding this comment.
This change updates the Go version used in the Dockerfile. This fixes the build failure and is a necessary change.
Right now the postsubmit build is failing: