Skip to content

Commit 7de74c2

Browse files
shai-almogclaude
andcommitted
ci: bump Metal screenshot step timeout from 30 to 45 minutes
The Metal screenshot pipeline hit the 30-minute step timeout on the latest PR run. Tracing the device-runner.log: the suite reached TabsTheme_light (~13 min into test execution), captured 111533 PNG bytes + 84124-byte preview chunk stream at preview_quality=6 (still over the 20480-byte preview cap), then went silent for ~18 minutes before the timer killed it. No FATAL / Test-failure markers - just dead air on the logcat replay. The earlier passing Metal run finished ~28 minutes in, so the suite is consistently running right at the wall. Bump the timeout to 45 minutes - matches the build-ios job's own cap and the iOS Metal runner's natural ceiling - so a borderline-slow chunk-stream replay doesn't get conflated with a real Metal port hang. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent d8b54f0 commit 7de74c2

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

.github/workflows/scripts-ios.yml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -408,7 +408,18 @@ jobs:
408408
"${{ steps.build-ios-app.outputs.workspace }}" \
409409
"" \
410410
"${{ steps.build-ios-app.outputs.scheme }}"
411-
timeout-minutes: 30
411+
# The Metal screenshot suite has been running right at the
412+
# 30-min wall on several recent PRs (the suite finishes ~28m
413+
# into the step). The latest run timed out around TabsTheme
414+
# after the runner appeared to stall on the preview-chunk
415+
# streaming for a particularly large capture (preview JPEG
416+
# blew past max_preview_bytes even at quality=6 and the
417+
# logcat replay went silent for ~18 minutes before the timer
418+
# fired). Bump to 45 minutes so we get headroom for the
419+
# logcat-buffer recovery path and don't conflate "Metal port
420+
# genuinely hung" with "step timer fired during a slow
421+
# chunk-stream replay" - the two need different responses.
422+
timeout-minutes: 45
412423

413424
- name: Publish Metal screenshot summary
414425
# Surfaces run-ios-ui-tests.sh's comparison result in the job's

0 commit comments

Comments
 (0)