Skip to content

Conversation

bryan-cox
Copy link
Contributor

@bryan-cox bryan-cox commented Sep 5, 2025

What type of PR is this?
/kind support

What this PR does / why we need it:

  • Update go.mod and hack/tools/go.mod to Go 1.24.7
  • Update Makefile GO_VERSION to 1.24.7
  • Update GitHub Actions workflow to use Go 1.24
  • Update golangci-kal.yml to use Go 1.24
  • Upgrade golangci-lint to v1.60.3 for Go 1.24 compatibility

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits
  • includes documentation
  • includes emoji in title
  • adds unit tests
  • adds or updates e2e tests

Release note:

Bump Go to v1.24

@k8s-ci-robot k8s-ci-robot added kind/support Categorizes issue or PR as a support question. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-priority labels Sep 5, 2025
@k8s-ci-robot k8s-ci-robot requested review from cnmcavoy and nrb September 5, 2025 12:33
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 5, 2025
@bryan-cox bryan-cox changed the title build: upgrade Go version to 1.24.7 🌱 build: upgrade Go version to 1.24.7 Sep 5, 2025
@bryan-cox bryan-cox changed the title 🌱 build: upgrade Go version to 1.24.7 🌱 Upgrade Go version to 1.24.7 Sep 5, 2025
@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Sep 5, 2025
@bryan-cox
Copy link
Contributor Author

/test pull-cluster-api-provider-aws-test

go.mod Outdated
module sigs.k8s.io/cluster-api-provider-aws/v2

go 1.23.1
go 1.24.7
Copy link
Member

Choose a reason for hiding this comment

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

With this we need to stay to go 1.24.0, so always to the 0 patch release to avoid importing issues from other components

module sigs.k8s.io/cluster-api-provider-aws/hack/tools

go 1.24.0
go 1.24.7
Copy link
Member

@damdo damdo Sep 5, 2025

Choose a reason for hiding this comment

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

@damdo
Copy link
Member

damdo commented Sep 5, 2025

Given the changes I suggested I would call this PR and the release notes

Bump Go to v1.24

@damdo
Copy link
Member

damdo commented Sep 5, 2025

@damdo
Copy link
Member

damdo commented Sep 5, 2025

These two also need updating:

For these you can use

  - name: 'gcr.io/k8s-staging-test-infra/gcb-docker-gcloud@sha256:63840f133e0dfeea0af9ef391210da7fab9d2676172e2967fccab0cd6110c4e7' # v20250513-9264efb079 go 1.24.3

Ref: https://github.com/kubernetes-sigs/cluster-api/blob/eb4e38c46ff6fb1d2124c0a8107f342a73b5bfae/cloudbuild-nightly.yaml#L8C135-L8C155


# Go
GO_VERSION ?=1.23.9
GO_VERSION ?=1.24.7
Copy link
Member

Choose a reason for hiding this comment

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

This can stay as is :)

- Update go.mod and hack/tools/go.mod to Go 1.24.7
- Update Makefile GO_VERSION to 1.24.7
- Update GitHub Actions workflow to use Go 1.24
- Update golangci-kal.yml to use Go 1.24
- Upgrade golangci-lint to v1.60.3 for Go 1.24 compatibility
@damdo damdo changed the title 🌱 Upgrade Go version to 1.24.7 🌱 Upgrade Go version to 1.24 Sep 5, 2025
Copy link
Member

@damdo damdo left a comment

Choose a reason for hiding this comment

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

Awesome thanks a lot @bryan-cox

/lgtm

/assign @nrb @richardcase @AndiDog @dlipovetsky

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

LGTM label has been added.

Git tree hash: 28ee518f5975a6afbab9eff3dff946bffd07a153

@richardcase
Copy link
Member

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: richardcase

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

The pull request process is described 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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 5, 2025
@k8s-ci-robot k8s-ci-robot merged commit c02551b into kubernetes-sigs:main Sep 5, 2025
21 checks passed
@bryan-cox bryan-cox deleted the bump-golang branch September 5, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/support Categorizes issue or PR as a support question. lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-priority release-note Denotes a PR that will be considered when it comes time to generate release notes. 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.

7 participants