Repository navigation
ci: add timeout-minutes to the cloud jobs and explain the branch input - #1298
Conversation
Without a limit, a hung cloud job runs until the GitHub default of 6 hours. Green runs take 2-3 minutes (desktop, OCR) and 5-17 minutes (Android, iOS, Sauce app); failing checks with retries took up to 34 minutes this week. Limits: 20 minutes for desktop and OCR, 45 for the mobile jobs, 5 for the schedule gate. The branch input of scheduled-tests now says how to run a PR from a fork (refs/pull/<number>/head) and that the run gets the cloud credentials, so the commit must be reviewed first. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
|
| inputs: | ||
| branch: | ||
| description: "Branch to run tests against" | ||
| description: "Branch of this repository, or refs/pull/<number>/head for a PR from a fork (only after reviewing that commit: the run gets the cloud credentials)" |
There was a problem hiding this comment.
The new instructions recommend refs/pull/<number>/head, but that ref can change after review. If the contributor pushes again before a job checks out the code, the job can run an unreviewed commit with cloud credentials. Recommend entering the reviewed commit SHA instead of the moving PR ref.
How this was verified: Each scheduled job checks out inputs.branch and then runs that checkout’s test commands with LambdaTest or Sauce credentials.
| description: "Branch of this repository, or refs/pull/<number>/head for a PR from a fork (only after reviewing that commit: the run gets the cloud credentials)" | |
| description: "Branch of this repository, or the full reviewed commit SHA for a PR from a fork (not refs/pull/<number>/head, which can change: the run gets the cloud credentials)" |
Summary
Adds
timeout-minutesto every job of the cloud workflows (e2e.yml,scheduled-tests.yml), like #1277 did forchecks.yml, and makes thebranchinput ofscheduled-testsexplain how to run a PR from a fork.Timeouts
Without a limit, a hung cloud job runs until the GitHub default of 6 hours (and keeps cloud sessions busy). Limits from the job durations of the recent runs:
Branch input
Before: "Branch to run tests against". After: "Branch of this repository, or refs/pull//head for a PR from a fork (only after reviewing that commit: the run gets the cloud credentials)". A fork branch name like
owner:branchdoes not work (the checkout failed with "invalid refspec" on 2026-10-08), and fork code must be reviewed before it gets the credentials.Test
🤖 Generated with Claude Code