-
Notifications
You must be signed in to change notification settings - Fork 321
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
Upgrade to Go 1.19 #1325
Upgrade to Go 1.19 #1325
Conversation
@@ -4,7 +4,4 @@ | |||
export DEBIAN_FRONTEND=noninteractive | |||
export TZ=America/Los_Angeles | |||
|
|||
apt-get update -qq && apt-get upgrade -qq |
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.
In .buildkite/test-rayjob-sample-yamls.yml
, a curl
command is executed before install-docker.sh
, so there's no need to install curl again.
@@ -60,7 +60,9 @@ | |||
|
|||
# Install python 3.10 and pip | |||
- apt-get update | |||
- apt-get install -y python3.10 python3-pip |
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.
apt-get install -y python3-pip
will report "error: externally-managed-environment" after I upgrade the image to golang:1.19
.
cc @anishasthana @blublinsky would you mind taking a look? Thx! |
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
@@ -1,6 +1,6 @@ | |||
module github.com/ray-project/kuberay/apiserver | |||
|
|||
go 1.17 | |||
go 1.19 |
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.
@kevin85421 may need to update the go version in the table https://github.com/ray-project/kuberay/blob/master/apiserver/DEVELOPMENT.md#kuberay-api-server-user-guide
Upgrade to Go 1.19
Upgrade to Go 1.19
Upgrade to Go 1.19
Upgrade to Go 1.19
Upgrade to Go 1.19
Upgrade to Go 1.19
Upgrade to Go 1.19
Upgrade to Go 1.19
Why are these changes needed?
crd:trivialVersions
is deprecated in controller-tool v0.7.0. See ⚠ bump to kubernetes 1.22.1, remove support for legacy v1beta1 CRDs and webhooks kubernetes-sigs/controller-tools#607 for more details.go.mod
/go.sum
: Updategolang:1.17
togolang:1.19
ingo.mod
, and then run thego mod tidy
command to add missing modules and remove unused ones.Related issue number
Checks