Skip to content

🐛 gracefully shutdown reconcilers and catalogd FBC server #4875

🐛 gracefully shutdown reconcilers and catalogd FBC server

🐛 gracefully shutdown reconcilers and catalogd FBC server #4875

Triggered via pull request February 2, 2025 04:32
Status Failure
Total duration 1m 58s
Artifacts

sanity.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 errors
verify: internal/util/context/context_test.go#L48
the cancel function returned by context.WithDeadline should be called, not discarded, to avoid a context leak
verify: internal/util/context/context_test.go#L60
the cancel function returned by context.WithCancel should be called, not discarded, to avoid a context leak
verify
Process completed with exit code 2.
lint: internal/util/context/context_test.go#L38
S1012: should use `time.Since` instead of `time.Now().Sub` (gosimple)
lint: internal/util/context/context_test.go#L37
S1024: should use time.Until instead of t.Sub(time.Now()) (gosimple)
lint: internal/util/context/context_test.go#L48
lostcancel: the cancel function returned by context.WithDeadline should be called, not discarded, to avoid a context leak (govet)
lint: internal/util/context/context_test.go#L60
lostcancel: the cancel function returned by context.WithCancel should be called, not discarded, to avoid a context leak (govet)
lint: internal/util/context/context_test.go#L96
SA1029: should not use built-in type string as key for value; define your own type to avoid collisions (staticcheck)
lint: internal/util/context/context_test.go#L36
require-error: for error assertions use require (testifylint)
lint: internal/util/context/context_test.go#L67
unnecessary trailing newline (whitespace)
lint
Process completed with exit code 2.