Skip to content

ci: fix Playwright runner timeout and automate NCDD pin updates - #9154

Merged
vitormattos merged 4 commits into
mainfrom
chore/sync-playwright-1.64.1-runtime
Oct 22, 2026
Merged

vitormattos merged 4 commits into
mainfrom
chore/sync-playwright-1.64.1-runtime

Conversation

@vitormattos

@vitormattos vitormattos commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Fix the Playwright CI runner timeout by moving E2E execution to the Ubuntu 24.04 host runner, which already provides PHP 8.3, instead of installing PHP inside the browser image.
  • Install Chromium for the Playwright version already resolved by package-lock.json (@playwright/test@1.64.1) using npx playwright install --with-deps chromium.
  • Update SMTP/API access to Mailpit via the host's mapped localhost ports.
  • Update the LibreSign devcontainer NCDD pin to 3d1fad2bb1bc9c880c43b68ea0845beda05c3400 (build(deps): Bump playwright from v1.63.0-noble to v1.64.1-noble LibreCodeCoop/nextcloud-docker-development#163); the NCDD Docker Playwright image remains pinned to 1.64.1.
  • Configure Renovate's custom github-digest manager to propose weekly reviews of new NCDD main-branch revisions. Existing Dependabot managers remain unchanged; no automerge.

Failure reproduced in CI

The previous Playwright run 39799347959 hit the 60-minute job timeout. The PHP setup step lasted approximately 50 minutes (15:17–16:07 UTC), leaving time for only 52 of 83 E2E tests, all of which passed. The Playwright summary then failed as a consequence of the cancellation.

For comparison, the same PHP setup took approximately 53 seconds on successful run 39513817337, which completed all 83 tests. Moving PHP setup onto the hosted Ubuntu runner avoids reinstalling that version in the browser container.

Merge dependency and operational note

Depends on LibreCodeCoop/nextcloud-docker-development#163. Its commit is pinned exactly for the devcontainer. Merge it before this PR; if the upstream PR is rebased or recreated, update the NCDD pin.

To get future automatic NCDD update PRs, Renovate must be enabled for LibreSign via its GitHub App or a self-hosted instance. Adding the configuration file alone does not start the bot. No automatic merge is configured.

Validation

  • Existing baseline: 83 E2E tests passed on Playwright run 39513817337.
  • Failure diagnosed from Playwright run 39799347959, including the slow PHP setup and 52 passing tests before cancellation.
  • New fix needs full CI validation on its current HEAD before merge; no green E2E result is claimed here.
  • All commits retain Conventional Commit formatting, DCO, and cryptographically verified signatures.

Please keep commits separate (no squash).

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos requested a review from a team as a code owner October 22, 2026 14:26
Pin the dependency to LibreCodeCoop/nextcloud-docker-development#163
so the local browser image matches the Playwright npm lockfile.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Use a regex manager and the GitHub digest datasource to propose
reviewed weekly updates of the pinned NCDD main branch commit.
Keep the existing Dependabot managers and disable Renovate automerge.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos force-pushed the chore/sync-playwright-1.64.1-runtime branch from 4d1d489 to ef1cbee Compare October 22, 2026 14:51
@vitormattos vitormattos changed the title ci: align Playwright CI and devcontainer with 1.64.1 build: sync Playwright 1.64.1 and automate NCDD pin updates Oct 22, 2026
Avoid slow PHP bootstrap inside the Playwright Docker image by using the
Ubuntu 24.04 runner with preinstalled PHP 8.3. Install Chromium matching
the npm lockfile and connect to Mailpit through mapped host ports.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos changed the title build: sync Playwright 1.64.1 and automate NCDD pin updates ci: fix Playwright runner timeout and automate NCDD pin updates Oct 22, 2026
@vitormattos
vitormattos merged commit d2e115b into main Oct 22, 2026
71 checks passed
@vitormattos
vitormattos deleted the chore/sync-playwright-1.64.1-runtime branch October 22, 2026 17:22
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