Skip to content

Conversation

@crazy-max
Copy link
Member

Signed-off-by: CrazyMax crazy-max@users.noreply.github.com

@crazy-max crazy-max marked this pull request as ready for review August 19, 2021 10:28
Dockerfile Outdated
FROM --platform=$BUILDPLATFORM tonistiigi/xx@sha256:21a61be4744f6531cb5f33b0e6f40ede41fa3a1b8c82d5946178f80cc84bfc04 AS xx

FROM --platform=$BUILDPLATFORM golang:1.16-alpine AS golatest
FROM --platform=$BUILDPLATFORM golang:1.17-alpine AS golatest
Copy link
Member

Choose a reason for hiding this comment

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

Should we have a test-matrices to test current and n-1 versions of go?

also see docker/cli#3258 (comment)

Copy link
Member Author

@crazy-max crazy-max Aug 19, 2021

Choose a reason for hiding this comment

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

Should we have a test-matrices to test current and n-1 versions of go?

Good idea I will add a dedicated matrix in a follow-up.

also see docker/cli#3258 (comment)

Ah yes buildx is rebuilt through the packaging pipeline. Forgot that.

Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure we need a matrix. This is a cli tool. Good pattern to leave them configurable though.

go.mod Outdated
golang.org/x/crypto v0.0.0-20210322153248-0c34fe9e7dc2 // indirect
golang.org/x/net v0.0.0-20210226172049-e18ecbb05110 // indirect
golang.org/x/oauth2 v0.0.0-20200107190931-bf48bf16ab8d // indirect
golang.org/x/sys v0.0.0-20210426230700-d19ff857e887 // indirect
Copy link
Collaborator

@AkihiroSuda AkihiroSuda Aug 19, 2021

Choose a reason for hiding this comment

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

Go 1.17 on macOS requires golang/sys@a76c4d0 (May 8) or later

@morlay
Copy link
Collaborator

morlay commented Aug 23, 2021

A extra topic, could we remove the whole vendor folder too?

the newer version of golang we use, the go modules is more stable.

@tonistiigi
Copy link
Member

A extra topic, could we remove the whole vendor folder too?

No, I'm against that. Dependencies should tracked as other source code for reviews, edits, debugging etc.

@crazy-max
Copy link
Member Author

@tonistiigi PTAL, I'll leave at 1.16 in go.mod for now to avoid golang/go#46144 in other projects.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@tonistiigi tonistiigi merged commit 6cf9fa8 into docker:master Sep 2, 2021
@crazy-max crazy-max deleted the go117 branch September 2, 2021 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants