Skip to content

modernize: Fix rangeint findings in datapath files#43248

Merged
aditighag merged 1 commit intomainfrom
pr/HadrienPatte/rangeint-datapath
Dec 11, 2025
Merged

modernize: Fix rangeint findings in datapath files#43248
aditighag merged 1 commit intomainfrom
pr/HadrienPatte/rangeint-datapath

Conversation

@HadrienPatte
Copy link
Copy Markdown
Member

@HadrienPatte HadrienPatte commented Dec 10, 2025

This PR addresses rangeint findings in datapath owned files in preparation to enabling this analyzer repository-wide (see here).

See #42642

@HadrienPatte
modernize: Fix rangeint findings in datapath files
326a59a 
This PR addresses
[`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint)
findings in datapath owned files in preparation to enabling this
analyzer repository-wide (see [here](https://github.com/cilium/cilium/blob/888c09ae6be6034d5c8a362659b7fd082f6b6a1b/.golangci.yaml#L119-L123)).

See #42642

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 10, 2025
@github-actions github-actions bot added the sig/policy Impacts whether traffic is allowed or denied based on user-defined policies. label Dec 10, 2025
@HadrienPatte
Copy link
Copy Markdown
Member Author

HadrienPatte commented Dec 10, 2025

/test

@HadrienPatte HadrienPatte marked this pull request as ready for review December 10, 2025 23:59
@HadrienPatte HadrienPatte requested review from a team as code owners December 10, 2025 23:59
@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Dec 11, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 11, 2025
@aditighag aditighag enabled auto-merge December 11, 2025 18:44
@aditighag aditighag added this pull request to the merge queue Dec 11, 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 Dec 11, 2025
Merged via the queue into main with commit 8cbf2c8 Dec 11, 2025
361 of 362 checks passed
@aditighag aditighag deleted the pr/HadrienPatte/rangeint-datapath branch December 11, 2025 20:12
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
073acba 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
@HadrienPatte HadrienPatte mentioned this pull request Feb 20, 2026
Merged
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
9ef64ee 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
1288256 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
c2ef143 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
67653ba 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
6847ba3 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
2432098 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
HadrienPatte added a commit that referenced this pull request Feb 20, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
68277a6 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
joestringer pushed a commit that referenced this pull request Feb 24, 2026
@HadrienPatte @joestringer
modernize: Enable rangeint analyzer
88196eb 
Followup to #43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
fzu-huang pushed a commit to fzu-huang/cilium that referenced this pull request Feb 25, 2026
@HadrienPatte
modernize: Enable rangeint analyzer
f76c014 
Followup to cilium#43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
javiercardona-work pushed a commit to javiercardona-work/cilium that referenced this pull request Mar 18, 2026
@HadrienPatte @javiercardona-work
modernize: Enable rangeint analyzer
4a1080a 
Followup to cilium#43248

Enable the [`rangeint`](https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/modernize#hdr-Analyzer_rangeint) modernize analyzer globally and address the last
remaining failures.

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bimmlerd bimmlerd bimmlerd approved these changes

@rgo3 rgo3 rgo3 approved these changes

@aditighag aditighag aditighag approved these changes

Assignees

No one assigned

Labels

ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/policy Impacts whether traffic is allowed or denied based on user-defined policies.

Projects

No open projects
cilium v1.19.0
Status: Released

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@HadrienPatte @bimmlerd @rgo3 @aditighag @tklauser