Skip to content

[release/1.4] Update Go to 1.15.13#5664

Merged
mikebrow merged 1 commit intocontainerd:release/1.4from
thaJeztah:1.4_update_golang
Jun 25, 2021
Merged

[release/1.4] Update Go to 1.15.13#5664
mikebrow merged 1 commit intocontainerd:release/1.4from
thaJeztah:1.4_update_golang

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

go1.15.13 (released 2021-06-03) includes security fixes to the archive/zip, math/big,
net, and net/http/httputil packages, as well as bug fixes to the linker, the go
command, and the math/big and net/http packages. See the Go 1.15.13 milestone on
the issue tracker for details:

https://github.com/golang/go/issues?q=milestone%3AGo1.15.13+label%3ACherryPickApproved

go1.15.12 (released 2021-05-06) includes a security fix to the net/http package,
as well as bug fixes to the runtime, the compiler, and the archive/zip, time,
and syscall packages. See the Go 1.15.12 milestone on the issue tracker for details:

https://github.com/golang/go/issues?q=milestone%3AGo1.15.12+label%3ACherryPickApproved

go1.15.13 (released 2021-06-03) includes security fixes to the archive/zip, math/big,
net, and net/http/httputil packages, as well as bug fixes to the linker, the go
command, and the math/big and net/http packages. See the Go 1.15.13 milestone on
the issue tracker for details:

https://github.com/golang/go/issues?q=milestone%3AGo1.15.13+label%3ACherryPickApproved

go1.15.12 (released 2021-05-06) includes a security fix to the net/http package,
as well as bug fixes to the runtime, the compiler, and the archive/zip, time,
and syscall packages. See the Go 1.15.12 milestone on the issue tracker for details:

https://github.com/golang/go/issues?q=milestone%3AGo1.15.12+label%3ACherryPickApproved

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
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
noting that our go.mod minimum version is currently set to go 1.16
https://github.com/containerd/containerd/blob/main/go.mod#L3

@thaJeztah
Copy link
Copy Markdown
Member Author

noting that our go.mod minimum version is currently set to go 1.16
https://github.com/containerd/containerd/blob/main/go.mod#L3

Yeah, that's the main branch, this is the release/1.4 branch, which is not yet using go modules. Overall, I think we should be "conservative" setting that to "latest" versions if not needed. Go currently only treats it as a warning, but it may at some point refuse to use the go module with other go versions (which may not be needed if the package someone uses does not require latest "go" version)

@thaJeztah
Copy link
Copy Markdown
Member Author

Is someone able to kick CI? (not sure why Windows failed, but probably just flaky)

@mikebrow
Copy link
Copy Markdown
Member

Yeah, that's the main branch, this is the release/1.4 branch, which is not yet using go modules. Overall, I think we should be "conservative" setting that to "latest" versions if not needed. Go currently only treats it as a warning, but it may at some point refuse to use the go module with other go versions (which may not be needed if the package someone uses does not require latest "go" version)

nod.. With respect to testing we should probably be presuming that those vendoring/building containerd will try to run on pretty much anything from 1.13 and up. https://github.com/containerd/containerd/blob/main/BUILDING.md#build-requirements same on main and release/1.4.

@mikebrow mikebrow merged commit 86f0c47 into containerd:release/1.4 Jun 25, 2021
@thaJeztah thaJeztah deleted the 1.4_update_golang branch June 25, 2021 16:57
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