Skip to content

Commit 03df85e

Browse files
author
Katie.Ransom
committed
Switch from seleniarm to selenium for arm64 architecture
The seleniarm fork has now been merged into the main selenium project.
1 parent e8d8ba4 commit 03df85e

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

runner/main/modules/docker-selenium/docker-selenium.sh

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -106,16 +106,14 @@ function docker-selenium_setup() {
106106
fi
107107

108108
# Only for testing purposes, we are going to introduce basic support for arm64 architecture.
109-
# Many of the docker images already are multi-arch, but not the selenium ones, that are using
110-
# a different repository for arm64. Let's use them here.
111109
# Important note: While we are still running Selenium 3, the arm64 images are using Selenium 4,
112110
# so they could come with new problems or incompatibilities.
113111
# As said, this is only for testing purposes, we are not going to support arm64 officially yet.
114112
# TODO: After testing, consider if we can make this the default for arm64, deleting this conf.
115113
# and using the seleniarm images when `uname -m` is arm64 or aarch64.
116114
if [[ -n ${TRY_SELENIARM} ]]; then
117-
chromeimage="seleniarm/standalone-chromium:latest"
118-
firefoximage="sseleniarm/standalone-firefox:latest"
115+
chromeimage="selenium/standalone-chromium:latest"
116+
firefoximage="selenium/standalone-firefox:latest"
119117
fi
120118

121119
# And these are the final images and options we will use.

0 commit comments

Comments
 (0)