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

🏗 Fix for recent changes to Chromedriver installation on CircleCI #39856

Merged

Conversation

danielrozenberg
Copy link
Member

@danielrozenberg danielrozenberg commented Feb 22, 2024

  • Update the circleci/browser-tools Orb to v1.4.8
  • Add an .npmrc file to e2e tasks with chromedriver_skip_download=true, since we're installing it with the Orb (and it currently fails for the same reason that the Orb's previous version failed - the download URL changed)
  • Use the Chrome version installed by CircleCI in amp visual-diff
  • Disabled a flaky test

@danielrozenberg danielrozenberg force-pushed the circleci-chromedriver-fix branch 3 times, most recently from afd2241 to 84c8c21 Compare February 22, 2024 16:25
@danielrozenberg danielrozenberg marked this pull request as ready for review February 22, 2024 16:28
@danielrozenberg danielrozenberg changed the title 🏗 Remove manual installation step of Chromedriver 🏗 Update orb circleci/browser-tools to v1.4.8 Feb 22, 2024
@danielrozenberg danielrozenberg enabled auto-merge (squash) February 22, 2024 16:29
ychsieh
ychsieh previously approved these changes Feb 22, 2024
@danielrozenberg danielrozenberg marked this pull request as draft February 22, 2024 16:55
auto-merge was automatically disabled February 22, 2024 16:55

Pull request was converted to draft

@danielrozenberg danielrozenberg force-pushed the circleci-chromedriver-fix branch 8 times, most recently from 1b24c7c to cea1deb Compare February 22, 2024 17:41
@danielrozenberg danielrozenberg changed the title 🏗 Update orb circleci/browser-tools to v1.4.8 🏗 Fix for recent changes to Chromedriver installation on CircleCI Feb 22, 2024
@danielrozenberg danielrozenberg force-pushed the circleci-chromedriver-fix branch 4 times, most recently from 8acdb9d to 553f536 Compare February 22, 2024 18:17
@danielrozenberg danielrozenberg force-pushed the circleci-chromedriver-fix branch 3 times, most recently from b4751a2 to 89ff197 Compare February 22, 2024 19:11
@danielrozenberg danielrozenberg force-pushed the circleci-chromedriver-fix branch 2 times, most recently from f6a6041 to bdba728 Compare February 22, 2024 19:33
@danielrozenberg danielrozenberg force-pushed the circleci-chromedriver-fix branch from bdba728 to a487954 Compare February 22, 2024 19:48
@danielrozenberg danielrozenberg dismissed ychsieh’s stale review February 22, 2024 20:11

Made big changes to this PR, probably should get this approved from scratch

@danielrozenberg danielrozenberg marked this pull request as ready for review February 22, 2024 20:11
@danielrozenberg danielrozenberg enabled auto-merge (squash) February 22, 2024 20:37
@danielrozenberg danielrozenberg merged commit 764f2f5 into ampproject:main Feb 22, 2024
43 checks passed
eszponder pushed a commit to krzysztofequativ/amphtml that referenced this pull request Apr 22, 2024
…pproject#39856)

* Fix for newly broken Chromedriver issues

* Use the Chrome version installed by CircleCI in `amp visual-diff`

* Disable flaky tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants