-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
[Infrastructure] Update Selenium versions #59268
Conversation
javiercn
commented
Dec 2, 2024
- Updates Selenium versions to 4.27.0
- Updates Docker image to 131.0
|
||
FROM mcr.microsoft.com/playwright/dotnet:v1.45.1-jammy-amd64 AS final | ||
FROM docker pull mcr.microsoft.com/playwright/dotnet:v1.46.0-noble AS final |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MackinnonBuck you updated this like 5 months ago.
Is it fair that we need to grab the latest version from https://hub.docker.com/r/microsoft/playwright-dotnet?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The version of the docker image needs to exactly match the version in Versions.props. Otherwise, the benchmarks will fail to run. So if we make this change, we should also update the Playwright dependency for the rest of the repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Is there a way this type of thing can be caught during the build? In CI i mean
Update Playwright version in Versions.props