-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release/1.4] Update Go to 1.16.14 #6527
Conversation
go1.16.13 (released 2022-01-06) includes fixes to the compiler, linker, runtime, and the net/http package. See the Go 1.16.13 milestone on our issue tracker for details: https://github.com/golang/go/issues?q=milestone%3AGo1.16.13+label%3ACherryPickApproved Signed-off-by: Sebastiaan van Stijn <[email protected]>
Not sure if there's gonna be more 1.4.x release, but if so, we may as well have the branch up to date |
Includes security fixes for crypto/elliptic (CVE-2022-23806), math/big (CVE-2022-23772), and cmd/go (CVE-2022-23773). go1.16.14 (released 2022-02-10) includes security fixes to the crypto/elliptic, math/big packages and to the go command, as well as bug fixes to the compiler, linker, runtime, the go command, and the debug/macho, debug/pe, net/http/httptest, and testing packages. See the Go 1.16.14 milestone on our issue tracker for details: https://github.com/golang/go/issues?q=milestone%3AGo1.16.14+label%3ACherryPickApproved full diff: golang/go@go1.16.13...go1.16.14 Signed-off-by: Sebastiaan van Stijn <[email protected]>
Just FYI, something has gotten stale on |
Ah, I see it indeed;
It guess it goes wrong here (as no version is specified); containerd/script/setup/install-critools Line 23 in 18a2715
Possibly something like 6b0e241 would be needed (trying to cherry-pick it, but doesn't apply clean) |
Build succeeded.
|
Pushed a commit, roughly based on 6b0e241 (#6328); let's see if that fixes it. Did a quick check, and ginkgo 1.16.5 looks to still have that flag; go install github.com/onsi/ginkgo/[email protected]
#...
ginkgo --help | grep skipMeasurements
-skipMeasurements |
Build succeeded.
|
Hmm... now it looks to be failing in vagrant provisioning
|
https://go.dev/doc/go-get-install-deprecation Based on commit 6b0e241, but modified to apply to the 1.4 branch. Signed-off-by: Kazuyoshi Kato <[email protected]> Signed-off-by: Sebastiaan van Stijn <[email protected]>
bceec34
to
d6d073b
Compare
Arf.. by bad; I used |
Build succeeded.
|
nice work; I had looked at that commit too; not sure how I decided that wasn't the source of the issue! :) I did see that recent ginko no longer has that flag but probably got distracted with other work before trying to resolve what version we were using! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Update Go to 1.16.14
Includes security fixes for crypto/elliptic (CVE-2022-23806), math/big (CVE-2022-23772),
and cmd/go (CVE-2022-23773).
go1.16.14 (released 2022-02-10) includes security fixes to the crypto/elliptic,
math/big packages and to the go command, as well as bug fixes to the compiler,
linker, runtime, the go command, and the debug/macho, debug/pe, net/http/httptest,
and testing packages. See the Go 1.16.14 milestone on our issue tracker for details:
https://github.com/golang/go/issues?q=milestone%3AGo1.16.14+label%3ACherryPickApproved
full diff: golang/go@go1.16.13...go1.16.14
Update Go to 1.16.13
go1.16.13 (released 2022-01-06) includes fixes to the compiler, linker, runtime,
and the net/http package. See the Go 1.16.13 milestone on our issue tracker for
details:
https://github.com/golang/go/issues?q=milestone%3AGo1.16.13+label%3ACherryPickApproved