Skip to content

Add CI timeouts and diagnostics#27

Merged
kvz merged 1 commit into
mainfrom
ci-timeouts
Jun 9, 2026
Merged

Add CI timeouts and diagnostics#27
kvz merged 1 commit into
mainfrom
ci-timeouts

Conversation

@kvz

@kvz kvz commented Jun 9, 2026

Copy link
Copy Markdown
Member

Why

PR #25 had to be admin-merged because unrelated Browser tests and Convex deployment QA jobs stayed in progress without enough diagnostics to tell whether Playwright install, Convex deploy, Vercel preview resolution, or cloud verification was stuck. The first run of this PR confirmed the old opaque hang: Playwright's Chromium installer reached 100% download and never returned.

What changed

  • Added job-level timeouts to the browser and Convex QA jobs.
  • Added step-level timeout wrappers around local verification, Convex deploy, Vercel preview resolution, and cloud verification.
  • Added command-level timeout support to the QA run() helper.
  • Added bounded timeouts around package packing, fixture install, Convex deploy, Convex env updates, browser build, and e2e tests.
  • Added stderr diagnostics for Vercel preview URL resolution while preserving stdout as the resolved URL.
  • Avoided the flaky Playwright Chromium installer in GitHub-hosted CI by using the runner's installed Chrome channel for e2e runs. Local/default runs still use Playwright's bundled Chromium unless PLAYWRIGHT_CHROMIUM_CHANNEL is set.

Validation

  • git diff --check
  • Ruby YAML parse for .github/workflows/*.{yml,yaml}
  • yarn check

Closes #26

@vercel

vercel Bot commented Jun 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
convex Ready Ready Preview, Comment Jun 9, 2026 9:06am

Request Review

@kvz kvz merged commit 3c61e3d into main Jun 9, 2026
11 checks passed
@kvz kvz deleted the ci-timeouts branch June 9, 2026 09:08
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.

Add timeouts and diagnostics for long-running CI jobs

1 participant