Skip to content

fix(ci): run Playwright E2E on GitHub Ubuntu - #40986

Merged
lyzno1 merged 3 commits into
mainfrom
fix/playwright-install-timeout
Aug 20, 2026
Merged

fix(ci): run Playwright E2E on GitHub Ubuntu#40986
lyzno1 merged 3 commits into
mainfrom
fix/playwright-install-timeout

Conversation

@WH-2099

@WH-2099 WH-2099 commented Aug 20, 2026

Copy link
Copy Markdown
Member

Summary

  • Run the Playwright full-stack E2E job on the GitHub Ubuntu 24.04 image.
  • Keep the standard Playwright system-dependency installation and the existing 15-minute step bound.
  • Avoid the Depot apt mirror that prevented browsers from being installed before any scenario ran.

Evidence

  • Depot timed out once at 15 minutes and again at 30 minutes while fetching Ubuntu packages at roughly 22 KB/s.
  • Skipping system dependencies installed both browser shells in 7 seconds and the Chromium suite passed, proving the test harness itself is healthy; WebKit then correctly reported its required GTK and GStreamer libraries.
  • The GitHub Ubuntu runner provides the supported apt environment without maintaining a repository-owned package list or mirror rewrite.

Validation

  • The final diff is one workflow line.
  • git diff --check passes.
  • Full-Stack E2E CI is the behavioral gate for Chromium and WebKit.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 20, 2026
@WH-2099 WH-2099 self-assigned this Aug 20, 2026
@github-actions github-actions Bot added the e2e End-to-end tests and E2E test infrastructure. label Aug 20, 2026
hjlarry
hjlarry previously approved these changes Aug 20, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 20, 2026
@lyzno1

lyzno1 commented Aug 20, 2026

Copy link
Copy Markdown
Member

We should ask the upstream Depot team for a proper fix, or switch back to the native runner. Increasing the timeout would only mask the underlying issue. Taking 30 minutes to download Playwright is clearly unacceptable.

@hjlarry

hjlarry commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

30m also not work...

@WH-2099
WH-2099 requested a review from lyzno1 as a code owner August 20, 2026 02:18
@WH-2099 WH-2099 changed the title fix(ci): extend Playwright install timeout fix(ci): avoid redundant Playwright dependency installs Aug 20, 2026
@WH-2099 WH-2099 changed the title fix(ci): avoid redundant Playwright dependency installs fix(ci): run Playwright E2E on GitHub Ubuntu Aug 20, 2026
@lyzno1
lyzno1 added this pull request to the merge queue Aug 20, 2026
Merged via the queue into main with commit ae20d64 Aug 20, 2026
37 checks passed
@lyzno1
lyzno1 deleted the fix/playwright-install-timeout branch August 20, 2026 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e End-to-end tests and E2E test infrastructure. lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants