Skip to content

only pop from queue once #4572

only pop from queue once

only pop from queue once #4572

Triggered via push August 15, 2024 14:28
Status Success
Total duration 2m 4s
Artifacts

validate.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

40 errors and 1 warning
[golangci] runtime/docker/build.go#L13: runtime/docker/build.go#L13
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] runtime/docker/build.go#L21: runtime/docker/build.go#L21
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] runtime/docker/build.go#L29: runtime/docker/build.go#L29
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] internal/context/context_test.go#L132: internal/context/context_test.go#L132
if statements should only be cuddled with assignments (wsl)
[golangci] internal/context/context_test.go#L148: internal/context/context_test.go#L148
assignments should only be cuddled with other assignments (wsl)
[golangci] internal/context/context_test.go#L149: internal/context/context_test.go#L149
only one cuddle assignment allowed before if statement (wsl)
[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.go#L709: executor/linux/build.go#L709
Function 'loadLazySecrets' is too long (164 > 160) (funlen)
[golangci] executor/linux/build_test.go#L1928: executor/linux/build_test.go#L1928
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] executor/linux/build_test.go#L212: executor/linux/build_test.go#L212
if statements should only be cuddled with assignments (wsl)
[golangci] executor/linux/build_test.go#L1526: executor/linux/build_test.go#L1526
if statements should only be cuddled with assignments used in the if statement itself (wsl)
[golangci] executor/linux/build_test.go#L1810: executor/linux/build_test.go#L1810
ranges should only be cuddled with assignments used in the iteration (wsl)
[golangci] executor/linux/build_test.go#L1820: executor/linux/build_test.go#L1820
if statements should only be cuddled with assignments (wsl)
[golangci] executor/linux/build_test.go#L2674: executor/linux/build_test.go#L2674
if statements should only be cuddled with assignments used in the if statement itself (wsl)
[golangci] executor/linux/build_test.go#L1669: executor/linux/build_test.go#L1669
defer statements should only be cuddled with expressions on same variable (wsl)
[golangci] executor/linux/build_test.go#L1815: executor/linux/build_test.go#L1815
if statements should only be cuddled with assignments used in the if statement itself (wsl)
[golangci] executor/linux/build_test.go#L2585: executor/linux/build_test.go#L2585
defer statements should only be cuddled with expressions on same variable (wsl)
[golangci] executor/linux/build_test.go#L2605: executor/linux/build_test.go#L2605
only one cuddle assignment allowed before if statement (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#L1152: executor/linux/build_test.go#L1152
defer statements should only be cuddled with expressions on same variable (wsl)
[golangci] executor/linux/build_test.go#L1773: executor/linux/build_test.go#L1773
only one cuddle assignment allowed before range statement (wsl)
[golangci] executor/linux/build_test.go#L1172: executor/linux/build_test.go#L1172
only one cuddle assignment allowed before if statement (wsl)
[golangci] executor/linux/build_test.go#L1214: executor/linux/build_test.go#L1214
ranges should only be cuddled with assignments used in the iteration (wsl)
[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] 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] 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] 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
cyclomatic complexity 31 of func `(*Worker).exec` is high (> 30) (gocyclo)
[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#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#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] cmd/vela-worker/exec.go#L29: cmd/vela-worker/exec.go#L29
directive `//nolint:nilerr,funlen // ignore returning nil - don't want to crash worker` is unused for linter "nilerr" (nolintlint)
validate
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2