Skip to content

Bump Go to 1.26.1 and golangci-lint to 2.11.3 - #499

Merged
ntnn merged 6 commits into
kbind-dev:mainfrom
cnvergence:bump-go-ver
Mar 30, 2026
Merged

ntnn merged 6 commits into
kbind-dev:mainfrom
cnvergence:bump-go-ver

Conversation

@cnvergence

@cnvergence cnvergence commented Mar 20, 2026 •

Copy link
Copy Markdown
Member

Summary

This PR will update Go version to 1.26.1, fixing recent CVEs.
We also need to bump golangci-lint, as build version was 1.24 in the previous golangci-lint version, this required fixing linter issues across our codebase.

What Type of PR Is This?

Related Issue(s)

Fixes #

Release Notes

NONE

@cnvergence
cnvergence requested a review from a team as a code owner March 20, 2026 19:02
Comment thread go.mod Outdated
// go-build-version 1.25.4
go 1.24.0
// go-build-version 1.25.8
go 1.25.8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The minimum should always be the .0, or?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think it should be equal, this is used for go version checking script to ensure all versions are consistent

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The go-build-version should be the version we build with, the go version itself should just be the major version, we do it in kcp the same way (unless there's a security issue that requires us to use a later go version)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I see, then lets do it the same way as in the kcp

@cnvergence cnvergence changed the title Bump Go to 1.25.8 Bump Go to 1.26.1 and golangci-lint to 2.11.3 Mar 30, 2026
Comment thread .golangci.yaml
Comment thread pkg/konnector/controllers/cluster/serviceexport/serviceexport_reconcile.go Outdated
Comment thread test/e2e/framework/dex.go Outdated
ntnn
ntnn previously approved these changes Mar 30, 2026

@ntnn ntnn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm
/approve

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@ntnn
ntnn enabled auto-merge March 30, 2026 14:22
@ntnn
ntnn merged commit be1021f into kbind-dev:main Mar 30, 2026
12 of 13 checks passed
@cnvergence
cnvergence deleted the bump-go-ver branch March 30, 2026 15:24
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.

2 participants