Skip to content

Makefile: give test-qemu-arm64 a 30m timeout - #14

Merged
mauri870 merged 2 commits into
masterfrom
qemu-timeout
Nov 3, 2026
Merged

mauri870 merged 2 commits into
masterfrom
qemu-timeout

Conversation

@mauri870

@mauri870 mauri870 commented Nov 3, 2026

Copy link
Copy Markdown
Owner

In 2196e07 I added test-qemu-arm64 without a -timeout, so the root package gets go test's default of 10 minutes. On CI, the arm64 qemu step has taken anywhere from 237 to 438 seconds. The tip job of #12, which adds one more toolexec script, hit the limit:

panic: test timed out after 10m0s
	running tests:
		TestScripts/flag_precedence (2m3s)
		TestScripts/min_max_int (2m10s)

Nothing was stuck. The emulated suite is just slow, and runner speed varies. Locally under qemu, toolexec_pgo takes about as long as the other toolexec scripts, around 25 seconds of wall time with the scripts running in parallel.

This change passes -timeout=30m to the target. make test-qemu-arm64 passes locally with the change.

After this merges, #12 needs a rerun, or a rebase onto master.

In 2196e07, I added test-qemu-arm64 without a -timeout, so the root
package gets go test's default of 10m. Under qemu on a CI runner the
script suite has taken from about 4 to 7 minutes, and the tip job of
#12, which adds one more toolexec script, ran out at 10m with
flag_precedence and min_max_int still running. Nothing was stuck: the
emulated suite is just slow, and the runners vary. Apologies for the
oversight.

The target now passes -timeout=30m.
@mauri870
mauri870 enabled auto-merge (squash) November 3, 2026 21:19
@mauri870
mauri870 merged commit 8c14429 into master Nov 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant