Skip to content

ci(e2e): refresh apt before playwright --with-deps (ondrej PPA label change)#1902

Open
arifulhoque7 wants to merge 1 commit into
weDevsOfficial:developfrom
arifulhoque7:fix/e2e-apt-releaseinfo
Open

ci(e2e): refresh apt before playwright --with-deps (ondrej PPA label change)#1902
arifulhoque7 wants to merge 1 commit into
weDevsOfficial:developfrom
arifulhoque7:fix/e2e-apt-releaseinfo

Conversation

@arifulhoque7

@arifulhoque7 arifulhoque7 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

setup-php adds the ondrej/php PPA; its InRelease Label changed ("PPA for PHP" → redirect notice), so apt-get update fails with exit 100 and playwright install --with-deps dies ("Failed to install browsers"). This blocked the e2e run right after the wpuf-pro clone succeeded.

Fix: run apt-get update --allow-releaseinfo-change before the browser-deps install, only on Playwright cache miss (when --with-deps actually hits apt).

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Improved the end-to-end test workflow to better handle cached Playwright browser downloads by refreshing package sources when needed. This helps installation steps run more reliably in CI.

…change)

setup-php adds the ondrej/php PPA; its InRelease Label changed from 'PPA
for PHP' to a redirect notice, so apt-get update fails with exit 100 and
'playwright install --with-deps' dies ('Failed to install browsers'). Run
apt-get update --allow-releaseinfo-change first to acknowledge the change.
Only runs on Playwright cache miss (when --with-deps actually hits apt).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc4091e5-97ef-4026-94da-9376839be397

📥 Commits

Reviewing files that changed from the base of the PR and between e7c6e79 and a3ed24b.

📒 Files selected for processing (1)
  • .github/workflows/e2e-wpuf.yml

Walkthrough

Adds a conditional step to the Playwright e2e GitHub Actions workflow that refreshes Ubuntu apt sources via sudo apt-get update --allow-releaseinfo-change when the browser-binaries cache is missed, executed before installing Playwright's OS dependencies.

Changes

E2E Workflow Apt Fix

Layer / File(s) Summary
Conditional apt source refresh step
.github/workflows/e2e-wpuf.yml
Adds a step that runs sudo apt-get update --allow-releaseinfo-change conditionally on a cache miss, before installing Playwright's browser OS dependencies.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Suggested labels: Ready to Merge

Poem

A little hop, a little tweak,
Apt sources fresh, no cache to seek 🐇
Playwright browsers now install with cheer,
No stale labels lurking near,
Thump-thump! CI runs clear! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the CI fix to refresh apt before Playwright dependency installation due to the ondrej PPA label change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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