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
update release-1.23 to go1.19 #113983
update release-1.23 to go1.19 #113983
Conversation
5c69d8f
to
203d8ac
Compare
pushed to rebase to pick up #114264 and address a review comment from the 1.24 PR in this copy of the commit |
/lgtm |
LGTM label has been added. Git tree hash: d4b28a0fd50a04519e6d2c3b425ec105ba88f044
|
required pull-kubernetes-dependencies and pull-kubernetes-verify presubmits will go green once we switch presubmit CI to go1.19.4 (xref kubernetes/release#2832 and kubernetes/test-infra#28293) edit: they are now green 🎉 |
interestingly, the
all files in https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/azcore require go1.18+ I can't find any evidence of that job ever having passed, but it may start passing once kubernetes/test-infra#28293 merges and CI switches to go1.19 and that build succeeds edit: |
/retest |
all green, cc @kubernetes/release-managers 🎁 Merry Christmas 🎄 |
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, dchen1107, dims, liggitt, pohly, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
post-submits all look just as good as they did prior to this merge |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Picks commits (from #113956 on top of #113416) needed to update release-1.23 to go1.19.
go1.18 goes out of support in ~Feb 2023. go1.19 is supported through ~August 2023, which covers the remainder of the 1.23 releases.
Historically, we haven't been able to reliably update release branches to new go versions. However, the go1.17 → go1.19 transition only has three changes we needed to adapt to (ErrDot LookPath errors, GC tuning changes, and removal of sha1 support by default), all of which we can adjust in-process to avoid changing release-1.23 behavior (once we get a go version containing https://go-review.googlesource.com/c/go/+/445656)
This PR includes commits from:
Match go1.17 defaults for SHA-1 and GC
commit, which modifies the kube-apiservermain
function to preserve SHA-1 and GC behavior matching go1.17Adjust for os/exec changes in 1.19
commit, which adjusts to preserve existing behavior with the go1.19 change to error when locating binaries in the current directoryThe golang.org/x/net update also happens to pick up the fix for CVE-2022-41717.
Before this can merge, 1.23 CI would need to be updated to use go1.19, so that gofmt and dependency CI checks pass (xref kubernetes/release#2822)
Which issue(s) this PR fixes:
xref #112408
Special notes for your reviewer:
Does this PR introduce a user-facing change?