Skip to content

ci: fix iOS simulator boot timeout#845

Merged
thymikee merged 1 commit into
mainfrom
ci/fix-ios-boot-timeout
Jun 23, 2026
Merged

ci: fix iOS simulator boot timeout#845
thymikee merged 1 commit into
mainfrom
ci/fix-ios-boot-timeout

Conversation

@thymikee

Copy link
Copy Markdown
Member

Summary

Fixes the iOS simulator boot timeout wrapper merged with #842. The previous watchdog used a pre-created mktemp marker, so a successful boot could still be treated as a timeout. This replaces the background watchdog with a simple Perl alarm around simctl bootstatus.

Validation

Validated YAML parsing and the embedded bash script syntax locally. The prior failed iOS run showed bootstatus finished successfully before the marker bug forced a failure.

@github-actions

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.3 MB 1.3 MB 0 B
JS gzip 427.0 kB 427.0 kB 0 B
npm tarball 561.8 kB 561.8 kB 0 B
npm unpacked 1.9 MB 1.9 MB 0 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 30.9 ms 30.8 ms -0.2 ms
CLI --help 51.4 ms 50.0 ms -1.4 ms

Top changed chunks: no changes in the largest emitted chunks.

@thymikee

Copy link
Copy Markdown
Member Author

Reviewed the iOS boot timeout fix at dde1abc. The old mktemp marker path could report a timeout after a successful bootstatus; the new perl alarm wrapper preserves the real simctl bootstatus exit code and only treats SIGALRM/142 as timeout.

All required checks are green, including iOS smoke. No findings; merge-ready. Residual risk is limited to the standard shell 128 + SIGALRM exit-code convention and alarm behavior across exec, which the workflow run validates on the target runner.

@thymikee thymikee merged commit 19f73c8 into main Jun 23, 2026
20 checks passed
@thymikee thymikee deleted the ci/fix-ios-boot-timeout branch June 23, 2026 14:24
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-06-23 14:25 UTC

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