Skip to content

Use Playwright container image for CI e2e job - #1597

Merged
domoritz merged 3 commits into
masterfrom
copilot/use-playwright-docker-container
Jun 4, 2026
Merged

Use Playwright container image for CI e2e job#1597
domoritz merged 3 commits into
masterfrom
copilot/use-playwright-docker-container

Conversation

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Playwright setup in CI was spending significant time installing browsers and system dependencies at runtime. This PR switches the e2e workflow to the prebuilt Playwright container model used in Mosaic to reduce setup overhead.

  • CI workflow: run e2e in Playwright container

    • Updated the e2e job in .github/workflows/test.yml to run in:
      • mcr.microsoft.com/playwright:v1.56.1-noble
    • Set container runtime user to match the Mosaic pattern:
      • options: --user 1001
  • Removed redundant Playwright bootstrap

    • Dropped the explicit install step:
      • npx playwright install --with-deps chromium
    • Kept the existing test command unchanged:
      • npx playwright test --project=chromium --reporter=github,html
e2e:
  runs-on: ubuntu-latest
  container:
    image: mcr.microsoft.com/playwright:v1.56.1-noble
    options: --user 1001
  steps:
    - uses: actions/checkout@v6
    - uses: actions/setup-node@v6
      with:
        cache: 'npm'
        node-version: 24
    - run: npm ci
    - run: npx playwright test --project=chromium --reporter=github,html

Copilot AI linked an issue Jun 4, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Use playwright docker container for improved installation speed Use Playwright container image for CI e2e job Jun 4, 2026
Copilot AI requested a review from domoritz June 4, 2026 03:26
@domoritz

domoritz commented Jun 4, 2026

Copy link
Copy Markdown
Member

@copilot check the logs of the failing test

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

@copilot check the logs of the failing test

Checked the failing E2E logs: Playwright reported a container/runtime mismatch (current: mcr.microsoft.com/playwright:v1.60.0-noble, required: mcr.microsoft.com/playwright:v1.56.1-noble) and missing browser executable. I fixed this by updating the workflow container tag to v1.56.1-noble in commit ff556a8.

@domoritz
domoritz marked this pull request as ready for review June 4, 2026 13:06
@domoritz
domoritz merged commit b69cad4 into master Jun 4, 2026
4 checks passed
@domoritz
domoritz deleted the copilot/use-playwright-docker-container branch June 4, 2026 13:06
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.

Use playwright docker container

2 participants