test(CI): move go test timeout to justfile (#13927)

The go-test CI workflow specifies at 10m timeout.  However, if a test exceeds this timeout (for example if it is deadlocked) then CI terminates the job and the go test output doesn't give us any clues about which test specifically was running.  This makes it hard to identify what caused the failure.

We move the 10m timeout from the CI workflow into the `go-test` justfile recipe.  This causes `go test` to exit with a more informative error message once the timeout is reached, instead of being terminated by CI.

Signed-off-by: Alex Leong <alex@buoyant.io>

---------

Signed-off-by: Alex Leong <alex@buoyant.io>
This commit is contained in:
Alex Leong 2025-04-11 16:15:52 -07:00 committed by GitHub
parent b4bffda34b
commit afbd4f29ce
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions

View File

@ -49,7 +49,7 @@ jobs:
go-test:
needs: meta
if: needs.meta.outputs.changed == 'true'
timeout-minutes: 10
timeout-minutes: 30
runs-on: ubuntu-24.04
container: ghcr.io/linkerd/dev:v45-go
steps:

View File

@ -18,7 +18,7 @@ go-lint *flags:
golangci-lint run {{ flags }}
go-test:
LINKERD_TEST_PRETTY_DIFF=1 gotestsum -- -race -v -mod=readonly ./...
LINKERD_TEST_PRETTY_DIFF=1 gotestsum -- -race -v -mod=readonly --timeout 10m ./...
##
## Rust