Skip to content

Conversation

skitt
Copy link
Member

@skitt skitt commented Sep 18, 2025

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

This bumps to the newest version of Go used by the oldest still supported branch of k/k (1.31, which uses Go 1.22).

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

This might run into issues with golangci-lint which should be bumped too. It would be best to use the corresponding GHA instead of the separate container image.

Release note:

NONE

@k8s-ci-robot k8s-ci-robot added kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Sep 18, 2025
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 18, 2025
strategy:
matrix:
go-version: [1.20.x, 1.21.x]
go-version: [1.22.x, 1.25.x]
Copy link
Member Author

Choose a reason for hiding this comment

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

Should this test all the versions we care about?

Copy link
Member

Choose a reason for hiding this comment

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

I think bracketing oldest / newest is ~fine

The oldest supported branch of k/k is 1.31, which uses Go 1.22.

Signed-off-by: Stephen Kitt <[email protected]>
@skitt
Copy link
Member Author

skitt commented Sep 18, 2025

/cc @liggitt

@liggitt
Copy link
Member

liggitt commented Sep 18, 2025

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 18, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: liggitt, skitt
Once this PR has been reviewed and has the lgtm label, please assign deads2k for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@skitt
Copy link
Member Author

skitt commented Sep 19, 2025

/cc @dims

@k8s-ci-robot k8s-ci-robot requested a review from dims September 19, 2025 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants