Skip to content

Switch e2e screenshot browser to firefox-esr - #881

Merged
guysoft merged 1 commit into
develfrom
feature/firefox-e2e
Aug 3, 2026
Merged

Switch e2e screenshot browser to firefox-esr#881
guysoft merged 1 commit into
develfrom
feature/firefox-e2e

Conversation

@guysoft

@guysoft guysoft commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fixes the failing devel e2e-test (run 30759697193): Chrome 147+ no longer produces --headless --screenshot images in the e2e container, so the screenshot retry loop consumed the 30-min job timeout.
  • Replaces google-chrome-stable with firefox-esr (Mozilla Team PPA; base image is Ubuntu 24.04 Noble). Firefox's --screenshot CLI is stable and ESR pins a major version for ~12 months.
  • Pairs with the merged CustomPiOS helper update (CustomPiOS#275, now in ghcr.io/guysoft/custompios:devel): browser-helpers.sh prefers firefox-esr/firefox and falls back to Chrome. Validated locally: firefox-esr 140.13.0esr installs and --headless --no-remote --screenshot emits a real PNG.

Test plan

  • e2e-test job passes: OctoPrint wizard (HTTP 200 + CONFIG_WIZARD) verified.
  • Screenshot artifact produced via Firefox (no 30-min timeout).
  • No regression vs the pre-Chrome-147 green run.

Chrome 147+ fails to produce --headless --screenshot images in the
e2e container (Page.captureScreenshot regression), which made the
e2e-test job exhaust its 30-min polling window and fail. Replace
google-chrome-stable with firefox-esr from the Mozilla Team PPA
(base image is Ubuntu 24.04 Noble). The shared browser-helpers.sh
in ghcr.io/guysoft/custompios:devel now prefers firefox-esr/firefox
and falls back to Chrome, so this pairs with the CustomPiOS helper
update. Validated locally: firefox-esr 140.13.0esr installs and
--headless --no-remote --screenshot emits a real PNG.
@guysoft
guysoft merged commit ddfecdc into devel Aug 3, 2026
3 checks passed
@guysoft
guysoft deleted the feature/firefox-e2e branch August 3, 2026 23:02
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