Skip to content

Conversation

@vvoland
Copy link
Contributor

@vvoland vvoland commented Apr 16, 2025

This commit changes the golang image to one that uses the Go 1.23.8.

1.23-alpine3.21 is still on go 1.23.7 and current go.mod requires go1.23.8 which makes the make vendor fail with:

0.117 go: go.mod requires go >= 1.23.8 (running go 1.23.7; GOTOOLCHAIN=local)
0.119 Error: failed to get ["github.com/docker/cli@v28.1.0-rc.1"]: failed to execute 'go [get github.com/docker/cli@v28.1.0-rc.1]': failed to execute binary "go" with args [get github.com/docker/cli@v28.1.0-rc.1]: go: go.mod requires go >= 1.23.8 (running go 1.23.7; GOTOOLCHAIN=local)
0.119  *errors.errorString

Description

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

This commit changes the golang image to one that uses the Go 1.23.8.

1.23-alpine3.21 is still on go 1.23.7 and current `go.mod` requires
go1.23.8 which makes the `make vendor` fail with:

```
0.117 go: go.mod requires go >= 1.23.8 (running go 1.23.7; GOTOOLCHAIN=local)
0.119 Error: failed to get ["github.com/docker/cli@v28.1.0-rc.1"]: failed to execute 'go [get github.com/docker/cli@v28.1.0-rc.1]': failed to execute binary "go" with args [get github.com/docker/cli@v28.1.0-rc.1]: go: go.mod requires go >= 1.23.8 (running go 1.23.7; GOTOOLCHAIN=local)
0.119  *errors.errorString
```

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@netlify
Copy link

netlify bot commented Apr 16, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 36c34f6
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67ff974fb511e30008363aac
😎 Deploy Preview https://deploy-preview-22437--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@aevesdocker aevesdocker merged commit ced10b9 into docker:main Apr 16, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants