Skip to content

Commit b13d8ae

Browse files
committed
Revert "ci: bump Metal screenshot step timeout from 30 to 45 minutes"
This reverts commit 7de74c2.
1 parent 7de74c2 commit b13d8ae

1 file changed

Lines changed: 1 addition & 12 deletions

File tree

.github/workflows/scripts-ios.yml

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -408,18 +408,7 @@ jobs:
408408
"${{ steps.build-ios-app.outputs.workspace }}" \
409409
"" \
410410
"${{ steps.build-ios-app.outputs.scheme }}"
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
411+
timeout-minutes: 30
423412

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

0 commit comments

Comments
 (0)