Skip to content

CI: allow Go 1.13 for Docker/Moby compatibility#5431

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
AkihiroSuda:go113
Apr 29, 2021
Merged

CI: allow Go 1.13 for Docker/Moby compatibility#5431
crosbymichael merged 1 commit intocontainerd:masterfrom
AkihiroSuda:go113

Conversation

@AkihiroSuda
Copy link
Copy Markdown
Member

Docker/Moby still uses Go 1.13 for building containerd binaries.

Docker/Moby still uses Go 1.13 for building containerd binaries.

Signed-off-by: Akihiro Suda <[email protected]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 28, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread .github/workflows/ci.yml
go-version: ['1.16.3']
include:
# Go 1.13.x is still used by Docker/Moby
- go-version: '1.13.x'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

when 1.18 drops (generics) n-3 will become problematic..

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry what do you mean?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is a language change for go called generics. It is currently planned for v1.18... if/when that happens we'll have to decide if we allow the use of said feature or if we will forego using it to help out the people who are not moving up.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah, I was expecting generics to be in Go 2.0 👀

We will have to help Moby-related repos to migrate to the latest Go before that...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

My impression is there won't be an actual go2.0 release, and the "go 2" project is more about driving the language forwards through 1.x releases.

@crosbymichael crosbymichael merged commit 3200519 into containerd:master Apr 29, 2021
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.

6 participants