Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ubuntu-latest for Firefox CI tests #8671

Merged
merged 7 commits into from
Dec 11, 2024
Merged

Use ubuntu-latest for Firefox CI tests #8671

merged 7 commits into from
Dec 11, 2024

Conversation

hsubox76
Copy link
Contributor

@hsubox76 hsubox76 commented Dec 9, 2024

We locked the ubuntu image for Firefox tests 2 years ago: #6833 because the new image was having issues. Hopefully that's been fixed by now, trying ubuntu-latest.

Conclusion: Firefox installed with apt-get seems to cause issues on Ubuntu 22.04, perhaps because of some conflict with Ubuntu 22.04's own pre-installed Snap version of Firefox. This can be solved either by removing the Firefox install altogether or by installing with puppeteer instead, which seems to not cause a conflict. Although removing the install and relying on the image's pre-installed Firefox works fine, and Github seems to update their Ubuntu regularly so it always has the latest version of Firefox, I prefer to use puppeteer for explicit control over the version. Maybe we want to lock it to some specific version or test with a specific version in the future, and this sets us up to easily do that.

@hsubox76 hsubox76 requested a review from a team as a code owner December 9, 2024 22:27
Copy link

changeset-bot bot commented Dec 9, 2024

⚠️ No Changeset found

Latest commit: 4115828

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@hsubox76 hsubox76 marked this pull request as draft December 9, 2024 22:29
Copy link
Contributor

@DellaBitta DellaBitta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming tests work, of course!

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Dec 9, 2024

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Dec 9, 2024

@hsubox76
Copy link
Contributor Author

Locking it to ubuntu-20.04 until we can figure out why (hopefully it isn't that current and future versions of Firefox don't work with karma-firefox-launcher).

Unfortunately it apparently is that.

@hsubox76
Copy link
Contributor Author

Problem never seems to have been fixed: karma-runner/karma-firefox-launcher#245

Possible solution: microsoft/playwright#20692

@hsubox76
Copy link
Contributor Author

hsubox76 commented Dec 11, 2024

This is probably the problem? https://www.reddit.com/r/Ubuntu/comments/tolwrf/wtf_ubuntu_snaponly_enforced_for_firefox/

Ubuntu 22.04 has a "snap" version of Firefox built in, which doesn't seem to work well. As part of the workflow, we install Firefox fresh with apt-get but I think the command "firefox" still is pointing at the snap version and not at the one we just installed, maybe? That seems to be the command karma-firefox-launcher uses ("firefox").

@hsubox76 hsubox76 marked this pull request as ready for review December 11, 2024 18:04
@hsubox76 hsubox76 merged commit 76c3641 into main Dec 11, 2024
39 checks passed
@hsubox76 hsubox76 deleted the ch-firefox-update branch December 11, 2024 20:33
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.

3 participants