-
Notifications
You must be signed in to change notification settings - Fork 858
Update Supported Kubernetes to 1.29, 1.30, 1.31 #4024
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 Supported Kubernetes to 1.29, 1.30, 1.31 #4024
Conversation
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Build Failed 😭 Build Id: 1803aad5-0227-4b7b-8637-2fd8dd48cf50 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Build Failed 😭 Build Id: bddc206a-9952-4f27-a889-27b9d0741b62 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Build Failed 😭 Build Id: f9b11ebe-2b14-4d07-bbcf-71a4bd0d5d32 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
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.
The Kubernetes version should be at 1.30 (or 1.30.5 if the patch version is used). The client-go can remain at v0.31.
build/build-image/Dockerfile
Outdated
@@ -53,7 +53,7 @@ RUN go install golang.org/x/tools/cmd/goimports@latest && \ | |||
# | |||
# When this is raised to 1.29: bump the code-generator version on line 66 to the v0.30 tag. | |||
# When this is raised to >=1.30: delete line 66-67 and uncomment line 68 to stop pinning code-generator. | |||
ENV KUBERNETES_VER 1.29.7 | |||
ENV KUBERNETES_VER 1.31.0 |
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.
Kubernetes cluster version should be the middle version, this this case K8s 1.30. The test cluster is currently at patch version 1.30.5 so we should use that one.
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.
Yes, you are correct that the Kubernetes version should be 1.30.5 in the Dockerfile.
But, while executing this step "make gen-embedded-openapi" , I have encountered the following error "--
workdir=/go/src/agones.dev/agones/build/scripts/k8s-export-openapi agones-build:7c4fe85bb7
go run -mod=mod ./main.go
2024/10/30 14:07:30 Failed to start kubectl proxy: fork/exec /usr/local/bin/kubectl: exec format error
exit status 1
make: *** [Makefile:727: gen-embedded-openapi] Error 1"
However, It works fine when I switch the Kubernetes cluster version to 1.31.0 in the above Dockerfile. Could you please guide me on resolving this error to ensure compatibility with version 1.30.5, if version 1.30.5 must be mandatory.
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.
Interesting, for some reason https://storage.googleapis.com/kubernetes-release/release/v1.30.5/bin/linux/amd64/kubectl
does not exist. If we use 1.30.4 instead of 1.30.5 for both Kubernetes and Kubectl versions it should work.
Edit: The link for easier to read table of the available Google storage release versions: https://pantheon.corp.google.com/storage/browser/kubernetes-release/release
build/build-image/Dockerfile
Outdated
@@ -53,7 +53,7 @@ RUN go install golang.org/x/tools/cmd/goimports@latest && \ | |||
# | |||
# When this is raised to 1.29: bump the code-generator version on line 66 to the v0.30 tag. | |||
# When this is raised to >=1.30: delete line 66-67 and uncomment line 68 to stop pinning code-generator. |
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.
Please also follow this comment since we will be at >=1.30.
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.
Be sure to delete lines 53-55 and update lines 66-68 to:
RUN mkdir -p /go/src/k8s.io && cd /go/src/k8s.io && \
git clone -b kubernetes-${KUBERNETES_VER} --depth=3 https://github.com/kubernetes/code-generator.git
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Build Failed 😭 Build Id: 3cd5ccc4-3c99-4d3c-9984-6434a43df506 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 🥳 Build Id: 8e125d20-dea5-46ce-9b22-917b6bf10e00 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
Build Succeeded 🥳 Build Id: 982362fa-9fcf-4829-862a-98bf41769c7c The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
* Update Supported to 1.29, 1.30, 1.31 * nit * regenerate crd docs * revert ID * using 1.30.4 patch version --------- Co-authored-by: Mengye (Max) Gong <8364575+gongmax@users.noreply.github.com>
What type of PR is this?
kind cleanup
What this PR does / Why we need it:
Which issue(s) this PR fixes:#4018
working on #4018
Special notes for your reviewer: