Skip to content

tool: validator script for Azure NPM to cilium migration #14655

tool: validator script for Azure NPM to cilium migration

tool: validator script for Azure NPM to cilium migration #14655

Triggered via pull request January 30, 2025 01:03
Status Failure
Total duration 7m 45s
Artifacts

golangci.yaml

on: pull_request
Matrix: Lint
Fit to window
Zoom out
Zoom in

Annotations

40 errors and 4 warnings
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L9
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L10
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L262
paramTypeCombine: func(serviceSelector *metav1.LabelSelector, policyPodSelector *metav1.LabelSelector) bool could be replaced with func(serviceSelector, policyPodSelector *metav1.LabelSelector) bool (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L51
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L97
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L129
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L188
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L198
rangeValCopy: each iteration copies 592 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L223
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L234
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L9
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L10
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L262
paramTypeCombine: func(serviceSelector *metav1.LabelSelector, policyPodSelector *metav1.LabelSelector) bool could be replaced with func(serviceSelector, policyPodSelector *metav1.LabelSelector) bool (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L51
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L97
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L129
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L188
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L198
rangeValCopy: each iteration copies 592 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L223
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, ubuntu-latest): tools/azure-npm-to-cilium-validator.go#L234
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L9
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L10
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L262
paramTypeCombine: func(serviceSelector *metav1.LabelSelector, policyPodSelector *metav1.LabelSelector) bool could be replaced with func(serviceSelector, policyPodSelector *metav1.LabelSelector) bool (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L51
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L97
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L129
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L188
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L198
rangeValCopy: each iteration copies 592 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L223
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L234
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L9
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L10
dupImport: package is imported 2 times under different aliases on lines 9 and 10 (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L262
paramTypeCombine: func(serviceSelector *metav1.LabelSelector, policyPodSelector *metav1.LabelSelector) bool could be replaced with func(serviceSelector, policyPodSelector *metav1.LabelSelector) bool (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L51
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L97
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L129
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L188
rangeValCopy: each iteration copies 328 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L198
rangeValCopy: each iteration copies 592 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L223
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.23.x, windows-latest): tools/azure-npm-to-cilium-validator.go#L234
rangeValCopy: each iteration copies 368 bytes (consider pointers or indexing) (gocritic)
Lint (1.22.x, ubuntu-latest)
Restore cache failed: Dependencies file is not found in /home/runner/work/azure-container-networking/azure-container-networking. Supported file pattern: go.sum
Lint (1.23.x, ubuntu-latest)
Restore cache failed: Dependencies file is not found in /home/runner/work/azure-container-networking/azure-container-networking. Supported file pattern: go.sum
Lint (1.22.x, windows-latest)
Restore cache failed: Dependencies file is not found in D:\a\azure-container-networking\azure-container-networking. Supported file pattern: go.sum
Lint (1.23.x, windows-latest)
Restore cache failed: Dependencies file is not found in D:\a\azure-container-networking\azure-container-networking. Supported file pattern: go.sum