Skip to content
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

Bump to k8s 1.29 #366

Merged
merged 6 commits into from
Jun 20, 2024
Merged

Conversation

ericpromislow
Copy link
Contributor

@ericpromislow ericpromislow commented May 3, 2024

Related to: rancher/rancher#45351

@ericpromislow ericpromislow requested a review from a team as a code owner May 3, 2024 22:16
@ericpromislow ericpromislow marked this pull request as draft May 3, 2024 22:16
@ericpromislow ericpromislow force-pushed the 45351-bump-to-k8s-1.29 branch 2 times, most recently from 8434bc2 to 75659c5 Compare May 21, 2024 19:26
@ericpromislow ericpromislow changed the title [WIP] {Not working yet] Bump to k8s modules v0.29.3 (for k8s 1.29) Bump to k8s 1.29 May 21, 2024
@ericpromislow ericpromislow marked this pull request as ready for review May 21, 2024 19:27
nflynt
nflynt previously approved these changes Jun 12, 2024
@ericpromislow
Copy link
Contributor Author

.drone.yml was updated to curl down helm instead of relying on presence of zypper, so no need to base drone on an older rancher image.

Comment on lines +3 to +5
go 1.22.0

toolchain go1.22.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any particular reason why we want a specific version of the toolchain?

In the case we want to keep the toolchain explicit should we try to use the latest toolchain? E.g. rancher/rancher currently uses go1.22.3 and toolchain go1.22.4 is already available.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed on Slack we'll merge as is and then revisit in a future PR that should follow pretty quickly.

@tomleb tomleb merged commit 236a12f into rancher:release/v0.5 Jun 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants