Upgrade supported Kubernetes versions to 1.34, 1.35, 1.36 - #4664
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. |
|
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. |
igooch
left a comment
There was a problem hiding this comment.
One blocking item (the terraform rotation), a couple worth resolving, details inline.
| "1.34" = ["us-east1", "RAPID"] | ||
| "1.35" = ["us-west1", "RAPID"] | ||
| //"1.36" = ["us-west1", "RAPID"] | ||
| "1.36" = ["us-west1", "RAPID"] |
There was a problem hiding this comment.
Same two-phase issue as build/terraform/e2e/module.tf — 1.33 is removed in the same PR that adds 1.36.
There was a problem hiding this comment.
It looks like something got out of order here, as 1.35 and 1.36 should be in different regions. 1.36 should be us-central1 and 1.37 us-east1.
| # |___/ |___/ | ||
|
|
||
| grpc_release_tag = v1.80.0 | ||
| grpc_release_tag = v1.83.0 |
There was a problem hiding this comment.
v1.83.0 doesn't match google.golang.org/grpc v1.82.1 in go.mod. Please double check that the versions is consistent across the repo.
|
Did we delete the currently supported test clusters? I noticed #4669 failed on: Which is a problem, since that without that PR, CI is broken... if we need to do a quick fix, add back in these lines, but line it up with the 0.15.0 release of Doscy - it should work. |
|
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. |
|
/gcbrun |
|
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. |
|
Just a heads up - I put the fix in that should solve the issue with Docsy not being locked to 0.15.0 🤞🏻 |
|
@Sivasankaran25 - you'll need to fix your DCO commits. |
* Fix for CI issue blocking the building of the site (lock to docsy 0.15.0) * Run `go mod tidy` for simple-game-server Signed-off-by: Mark Mandel <mark@compoundtheory.com>
bd0eb8a to
d1d3b9f
Compare
|
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. |
|
Yay it passed! @Sivasankaran25 if you fix the DCO, then @igooch can wrap this up, and we can unstick CI 😄 |
igooch
left a comment
There was a problem hiding this comment.
Two additional small findings in line:
| require gopkg.in/yaml.v2 v2.4.0 | ||
|
|
||
| require ( | ||
| github.com/google/docsy v0.15.0 // indirect |
There was a problem hiding this comment.
Two small things: docsy is a direct dependency of the site build, so the // indirect markers look wrong, and the file is missing its trailing newline. A go mod tidy in site/ should fix both.
There was a problem hiding this comment.
Leave this as is please. It works and unblocks CI (and it's how we did it before). #4669 solves this properly.
| "1.34" = ["us-east1", "RAPID"] | ||
| "1.35" = ["us-west1", "RAPID"] | ||
| //"1.36" = ["us-west1", "RAPID"] | ||
| "1.36" = ["us-west1", "RAPID"] |
There was a problem hiding this comment.
It looks like something got out of order here, as 1.35 and 1.36 should be in different regions. 1.36 should be us-central1 and 1.37 us-east1.
|
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. |
What type of PR is this?
/kind breaking
What this PR does / Why we need it:
Which issue(s) this PR fixes:
Works On #4655
Did you use AI tools in preparing this PR?:
Y/N
Special notes for your reviewer: