-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
WIP: update release-1.23 to go1.18 #113416
Conversation
@liggitt: This cherry pick PR is for a release branch and has not yet been approved by Release Managers. To merge this cherry pick, it must first be approved ( AFTER it has been approved by code owners, please leave the following comment on a line by itself, with no leading whitespace: /cc kubernetes/release-managers (This command will request a cherry pick review from Release Managers and should work for all GitHub users, whether they are members of the Kubernetes GitHub organization or not.) For details on the patch release process and schedule, see the Patch Releases page. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@liggitt: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt 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 |
/remove-sig api-machinery |
This PR has the label work-in-progress, please revisit to see if you still need this, please close if not |
3223eae
to
5000c77
Compare
/remove-sig api-machinery |
Mimic commit 9329500
5000c77
to
887e5b7
Compare
golang.org/x/crypto v0.0.0-20220214200702-86341886e292 golang.org/x/net v0.3.1-0.20221206200815-1e63c2f08a10 golang.org/x/oauth2 v0.0.0-20211104180415-d3ed0bb246c8 golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4 golang.org/x/sys v0.3.0 golang.org/x/term v0.3.0 golang.org/x/time v0.0.0-20220210224613-90d013bbcef8 golang.org/x/tools v0.1.12 Mimic commit e6e7a42 with current release-1.24 versions
This should fix Go 1.18 compatibility woes. Cherry-pick of e3ed3ba with conflict resolved Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This fixes the following error: > test/integration/apimachinery/watch_restart_test.go:232:5: call to (*T).Fatalf from a non-test goroutine Update: the previous fix (commit 1ce55e3 that uses panic) looks way too severe; it seems it is enough to call t.Errorf and return from goroutine. The test will time out and fail anyway. Combination of 1ce55e3 and 8820f4d Co-authored-by: Kir Kolyshkin <kolyshkin@gmail.com> Co-authored-by: Davanum Srinivas <davanum@gmail.com>
Generated by ./hack/update-mocks.sh using go 1.18 Mimic of commit 4513de0
This should fix the following error when running ./hack/update-generated-stable-metrics.sh: 'go get' is no longer supported outside a module. To build and install a command, use 'go install' with a version, like 'go install example.com/cmd@latest' For more information, see https://golang.org/doc/go-get-install-deprecation or run 'go help get' or 'go help install'. Using `go get` to download gopkg.in/yaml.v2 package into KUBE_EXTRA_GOPATH directory no longer works. Interestingly, main repo already has gopkg.in/yaml.v2@v2.4.0, same version that was installed by that go get. I guess that GOPATH with multiple elements no longer works either, and since this code was the only user of KUBE_EXTRA_GOPATH, let's remove it as well. Combination of 3470453 and d8fdb00 Co-authored-by: Kir Kolyshkin <kolyshkin@gmail.com> Co-authored-by: Davanum Srinivas <davanum@gmail.com>
Replay of 9e1064a
Cherry-pick of 5d8f6f1
887e5b7
to
4c3c852
Compare
@liggitt: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/remove-sig api-machinery |
/close in favor of #113983 |
@liggitt: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Picks commits needed to update release-1.23 to go1.18.
go1.17 went out of support in Aug 2022. go1.18 is supported through ~Feb 2023.
Historically, we haven't been able to reliably update release branches to new go versions. However, the go1.17 → go1.18 transition only had two changes we needed to adapt to (GC tuning changes and removal of sha1 support by default), both 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:
The only commit new to this PR is the
Match go1.17 defaults for SHA-1 and GC
oneBefore this could merge:
Which issue(s) this PR fixes:
xref #112408
Special notes for your reviewer:
Does this PR introduce a user-facing change?
/hold