fix(patch/v25.2): apply worker status fix #4769
ecrupperAnnotations
59 errors
|
[golangci] runtime/kubernetes/mock.go#L143:
runtime/kubernetes/mock.go#L143
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
|
|
[golangci] runtime/kubernetes/pod_tracker.go#L226:
runtime/kubernetes/pod_tracker.go#L226
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
|
|
[golangci] cmd/vela-worker/exec.go#L31:
cmd/vela-worker/exec.go#L31
cyclomatic complexity 31 of func `(*Worker).exec` is high (> 30) (gocyclo)
|
|
[golangci] internal/service/snapshot_test.go#L138:
internal/service/snapshot_test.go#L138
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] internal/service/upload_test.go#L166:
internal/service/upload_test.go#L166
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] mock/docker/node.go#L22:
mock/docker/node.go#L22
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] mock/docker/node.go#L31:
mock/docker/node.go#L31
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] mock/docker/service.go#L24:
mock/docker/service.go#L24
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] runtime/kubernetes/pod_tracker_test.go#L184:
runtime/kubernetes/pod_tracker_test.go#L184
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] executor/local/build_test.go#L470:
executor/local/build_test.go#L470
unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] executor/local/build_test.go#L515:
executor/local/build_test.go#L515
unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] executor/linux/build_test.go#L1100:
executor/linux/build_test.go#L1100
unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] executor/linux/secret.go#L209:
executor/linux/secret.go#L209
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
|
|
[golangci] executor/linux/secret.go#L226:
executor/linux/secret.go#L226
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
|
|
[golangci] executor/linux/secret.go#L243:
executor/linux/secret.go#L243
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
|
|
[golangci] mock/docker/container.go#L127:
mock/docker/container.go#L127
SA1019: types.ContainerExecInspect is deprecated: use [container.ExecInspect]. (staticcheck)
|
|
[golangci] mock/docker/container.go#L388:
mock/docker/container.go#L388
SA1019: types.ContainerPathStat is deprecated: use [container.PathStat]. (staticcheck)
|
|
[golangci] mock/docker/container.go#L397:
mock/docker/container.go#L397
SA1019: types.ContainerStats is deprecated: use [container.StatsResponseReader]. (staticcheck)
|
|
[golangci] mock/docker/container.go#L495:
mock/docker/container.go#L495
SA1019: types.ContainersPruneReport is deprecated: use [container.PruneReport]. (staticcheck)
|
|
[golangci] mock/docker/container.go#L503:
mock/docker/container.go#L503
SA1019: types.ContainerStats is deprecated: use [container.StatsResponseReader]. (staticcheck)
|
|
[golangci] mock/docker/container.go#L511:
mock/docker/container.go#L511
SA1019: types.ContainerPathStat is deprecated: use [container.PathStat]. (staticcheck)
|
|
[golangci] mock/docker/network.go#L40:
mock/docker/network.go#L40
SA1019: types.NetworkCreateResponse is deprecated: use [network.CreateResponse]. (staticcheck)
|
|
[golangci] mock/docker/network.go#L47:
mock/docker/network.go#L47
SA1019: types.NetworkCreateResponse is deprecated: use [network.CreateResponse]. (staticcheck)
|
|
[golangci] mock/docker/network.go#L56:
mock/docker/network.go#L56
SA1019: types.NetworkCreateResponse is deprecated: use [network.CreateResponse]. (staticcheck)
|
|
[golangci] mock/docker/network.go#L84:
mock/docker/network.go#L84
SA1019: types.NetworkResource is deprecated: use [network.Inspect] or [network.Summary] (for list operations). (staticcheck)
|
|
[golangci] mock/docker/network.go#L89:
mock/docker/network.go#L89
SA1019: types.NetworkResource is deprecated: use [network.Inspect] or [network.Summary] (for list operations). (staticcheck)
|
|
[golangci] mock/docker/network.go#L96:
mock/docker/network.go#L96
SA1019: types.NetworkResource is deprecated: use [network.Inspect] or [network.Summary] (for list operations). (staticcheck)
|
|
[golangci] mock/docker/network.go#L176:
mock/docker/network.go#L176
SA1019: types.NetworksPruneReport is deprecated: use [network.PruneReport]. (staticcheck)
|
|
[golangci] mock/docker/volume.go#L173:
mock/docker/volume.go#L173
SA1019: types.VolumesPruneReport is deprecated: use [volume.PruneReport]. (staticcheck)
|
|
[golangci] mock/worker/executor.go#L38:
mock/worker/executor.go#L38
SA1019: library.Executor is deprecated: use Executor from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
[golangci] mock/worker/executor.go#L58:
mock/worker/executor.go#L58
SA1019: library.Executor is deprecated: use Executor from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
[golangci] runtime/docker/container.go#L303:
runtime/docker/container.go#L303
SA1019: types.ExecConfig is deprecated: use [container.ExecOptions]. (staticcheck)
|
|
[golangci] runtime/docker/container.go#L315:
runtime/docker/container.go#L315
SA1019: types.ExecStartCheck is deprecated: use [container.ExecStartOptions] or [container.ExecAttachOptions]. (staticcheck)
|
|
[golangci] runtime/docker/network.go#L23:
runtime/docker/network.go#L23
SA1019: types.NetworkCreate is deprecated: use [network.CreateOptions]. (staticcheck)
|
|
[golangci] runtime/docker/network.go#L45:
runtime/docker/network.go#L45
SA1019: types.NetworkInspectOptions is deprecated: use [network.InspectOptions]. (staticcheck)
|
|
[golangci] cmd/vela-worker/start.go#L71:
cmd/vela-worker/start.go#L71
if statements should only be cuddled with assignments (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L50:
cmd/vela-worker/start.go#L50
assignments should only be cuddled with other assignments (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L51:
cmd/vela-worker/start.go#L51
only one cuddle assignment allowed before if statement (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L54:
cmd/vela-worker/start.go#L54
expressions should not be cuddled with blocks (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L57:
cmd/vela-worker/start.go#L57
assignments should only be cuddled with other assignments (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L58:
cmd/vela-worker/start.go#L58
only one cuddle assignment allowed before if statement (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L61:
cmd/vela-worker/start.go#L61
return statements should not be cuddled if block has more than two lines (wsl)
|
|
[golangci] cmd/vela-worker/start.go#L70:
cmd/vela-worker/start.go#L70
expressions should not be cuddled with declarations or returns (wsl)
|
|
[golangci] executor/linux/build_test.go#L389:
executor/linux/build_test.go#L389
if statements should only be cuddled with assignments used in the if statement itself (wsl)
|
|
[golangci] executor/linux/build_test.go#L384:
executor/linux/build_test.go#L384
ranges should only be cuddled with assignments used in the iteration (wsl)
|
|
[golangci] executor/linux/build_test.go#L987:
executor/linux/build_test.go#L987
if statements should only be cuddled with assignments used in the if statement itself (wsl)
|
|
[golangci] executor/linux/build_test.go#L992:
executor/linux/build_test.go#L992
if statements should only be cuddled with assignments (wsl)
|
|
[golangci] executor/linux/build_test.go#L1595:
executor/linux/build_test.go#L1595
ranges should only be cuddled with assignments used in the iteration (wsl)
|
|
[golangci] executor/linux/build_test.go#L202:
executor/linux/build_test.go#L202
ranges should only be cuddled with assignments used in the iteration (wsl)
|
|
[golangci] executor/linux/build_test.go#L394:
executor/linux/build_test.go#L394
if statements should only be cuddled with assignments (wsl)
|
|
[golangci] executor/linux/build_test.go#L863:
executor/linux/build_test.go#L863
only one cuddle assignment allowed before if statement (wsl)
|
|
[golangci] executor/linux/build_test.go#L207:
executor/linux/build_test.go#L207
if statements should only be cuddled with assignments used in the if statement itself (wsl)
|
|
[golangci] executor/linux/build_test.go#L605:
executor/linux/build_test.go#L605
defer statements should only be cuddled with expressions on same variable (wsl)
|
|
[golangci] executor/linux/build_test.go#L945:
executor/linux/build_test.go#L945
only one cuddle assignment allowed before range statement (wsl)
|
|
[golangci] executor/linux/build_test.go#L1757:
executor/linux/build_test.go#L1757
defer statements should only be cuddled with expressions on same variable (wsl)
|
|
[golangci] executor/linux/build_test.go#L145:
executor/linux/build_test.go#L145
defer statements should only be cuddled with expressions on same variable (wsl)
|
|
[golangci] internal/context/context_test.go#L148:
internal/context/context_test.go#L148
assignments should only be cuddled with other assignments (wsl)
|
|
[golangci] runtime/kubernetes/pod_tracker_test.go#L373:
runtime/kubernetes/pod_tracker_test.go#L373
expressions should not be cuddled with blocks (wsl)
|
|
[golangci] cmd/vela-worker/exec.go#L30:
cmd/vela-worker/exec.go#L30
directive `//nolint:nilerr,funlen // ignore returning nil - don't want to crash worker` is unused for linter "nilerr" (nolintlint)
|