Skip to content

all: fix golangci-lint 2.6.0 govet errors#42520

Merged
tklauser merged 1 commit intomainfrom
pr/tklauser/golangci-lint-fix-2.6.0
Nov 3, 2025
Merged

all: fix golangci-lint 2.6.0 govet errors#42520
tklauser merged 1 commit intomainfrom
pr/tklauser/golangci-lint-fix-2.6.0

Conversation

@tklauser
Copy link
Copy Markdown
Member

Fix the following errors appearing when updating to golangci-lint 2.6.0:

2025-10-31T11:18:01.5671846Z ##[error]cilium-dbg/cmd/encrypt_status.go:5:1: buildtag: +build line is no longer needed (govet)
2025-10-31T11:18:01.5673091Z // +build linux
2025-10-31T11:18:01.5673532Z ^
2025-10-31T11:18:01.5674421Z ##[error]pkg/endpoint/bpf.go:587:25: printf: missing ... in args forwarded to printf-like function (govet)
2025-10-31T11:18:01.5675531Z 				e.getLogger().Debug(fmt.Sprintf(format, args))
2025-10-31T11:18:01.5676128Z 				                    ^
2025-10-31T11:18:01.5677278Z ##[error]pkg/metrics/plot.go:128:26: printf: non-constant format string in call to writeText (govet)
2025-10-31T11:18:01.5678555Z 	writeText(0, originX-1, "╭"+strings.Repeat("─", width-originX-1)+"╮")
2025-10-31T11:18:01.5678972Z 	                        ^
2025-10-31T11:18:01.5679670Z ##[error]pkg/metrics/plot.go:143:26: printf: non-constant format string in call to writeText (govet)
2025-10-31T11:18:01.5680636Z 	writeText(8, originX-1, "╰"+strings.Repeat("─", width-originX-1)+"╯")

(from https://github.com/cilium/cilium/actions/runs/18970710287/job/54177614867?pr=42514)

Fix the following errors appearing when updating to golangci-lint 2.6.0:

```
2025-10-31T11:18:01.5671846Z ##[error]cilium-dbg/cmd/encrypt_status.go:5:1: buildtag: +build line is no longer needed (govet)
2025-10-31T11:18:01.5673091Z // +build linux
2025-10-31T11:18:01.5673532Z ^
2025-10-31T11:18:01.5674421Z ##[error]pkg/endpoint/bpf.go:587:25: printf: missing ... in args forwarded to printf-like function (govet)
2025-10-31T11:18:01.5675531Z 				e.getLogger().Debug(fmt.Sprintf(format, args))
2025-10-31T11:18:01.5676128Z 				                    ^
2025-10-31T11:18:01.5677278Z ##[error]pkg/metrics/plot.go:128:26: printf: non-constant format string in call to writeText (govet)
2025-10-31T11:18:01.5678555Z 	writeText(0, originX-1, "╭"+strings.Repeat("─", width-originX-1)+"╮")
2025-10-31T11:18:01.5678972Z 	                        ^
2025-10-31T11:18:01.5679670Z ##[error]pkg/metrics/plot.go:143:26: printf: non-constant format string in call to writeText (govet)
2025-10-31T11:18:01.5680636Z 	writeText(8, originX-1, "╰"+strings.Repeat("─", width-originX-1)+"╯")
```

Signed-off-by: Tobias Klauser <[email protected]>
@tklauser tklauser requested review from a team as code owners October 31, 2025 12:19
@tklauser tklauser added the area/CI Continuous Integration testing issue or flake label Oct 31, 2025
@tklauser tklauser requested a review from derailed October 31, 2025 12:19
@tklauser tklauser added the release-note/ci This PR makes changes to the CI. label Oct 31, 2025
@tklauser
Copy link
Copy Markdown
Member Author

/test

@tklauser tklauser enabled auto-merge October 31, 2025 12:20
Copy link
Copy Markdown
Member

@fristonio fristonio left a comment

Choose a reason for hiding this comment

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

🚀

@tklauser tklauser added this pull request to the merge queue Nov 3, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 3, 2025
Merged via the queue into main with commit b70e592 Nov 3, 2025
461 of 466 checks passed
@tklauser tklauser deleted the pr/tklauser/golangci-lint-fix-2.6.0 branch November 3, 2025 23:04
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.

Projects

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

5 participants