-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
enable go version update using the check_install_golang script #20278
base: master
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: prezha 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 |
/ok-to-test |
kvm2 driver with docker runtime
Times for minikube start: 51.2s 51.4s 51.7s 50.9s 51.1s Times for minikube ingress: 18.5s 15.1s 18.5s 18.6s 19.6s docker driver with docker runtime
Times for minikube start: 24.5s 24.6s 21.2s 21.0s 24.7s Times for minikube ingress: 12.3s 12.8s 12.3s 12.8s 12.8s docker driver with containerd runtime
Times for minikube start: 19.9s 19.3s 22.2s 21.9s 22.5s Times for minikube ingress: 22.8s 22.8s 22.8s 38.8s 38.8s |
Here are the number of top 10 failed tests in each environments with lowest flake rate.
Besides the following environments also have failed tests:
To see the flake rates of all tests by environment, click here. |
@@ -1,8 +1,6 @@ | |||
module k8s.io/minikube | |||
|
|||
go 1.23.0 | |||
|
|||
toolchain go1.23.4 |
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.
would it worth to use toolchain local?
we've hit an issue with updating the go version on jenkins ci runners, which was required to fix another issue with the
make generate-licenses
, blocking our releasethe details are in a rather lengthy slack debugging thread, but here's the tldr:
so, the issue is probably in the
toolchain go1.23.4
line that we have set in thego.mod
file and that's what go uses to respond back ongo version
, regardless of the actual go version installed (which explains why, when you start another-ssh session, you probably don't go into the minikube source dir, so thego version
is not affected by the toolchain directive and returns the actually installed go version, and so you were able to update the go version as expected by manually running the script)the "proof" whilst having
go v1.23.3
installed:going outside the minikube src dir:
going back to the minikube src and removing the toolchain go1.23.4:
that's probably "by design", but we oversaw it
this pr removes the toolchain directive and also bumps the go version in the go.mod, as for some reason, looks like the other pr did not do it (worth checking separatelly)