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

Support fort Selenium 2.53.1 #171

Closed
pavelpp opened this issue Jul 13, 2016 · 4 comments
Closed

Support fort Selenium 2.53.1 #171

pavelpp opened this issue Jul 13, 2016 · 4 comments
Assignees
Milestone

Comments

@pavelpp
Copy link
Contributor

pavelpp commented Jul 13, 2016

When is this going to be added? Currently I'm getting org.testcontainers.containers.ContainerFetchException: Retry limit reached while trying to pull image: selenium/standalone-chrome-debug:2.53.1

@rnorth
Copy link
Member

rnorth commented Jul 16, 2016

Hi
The selenium docker images are maintained by the SeleniumHQ team, not by
us. If you're using a newer version of the selenium Java libs than there is
a container image for you'd see this issue.

I'd imagine they'd appreciate a PR in case the missing 2.53.1 is just a
simple omission.

In the meantime can you fall back to a version of selenium for which the
docker image is also available?

@dmart28
Copy link
Contributor

dmart28 commented Sep 5, 2016

Hi, just curious - why not give an easy way to change version directly with BrowserWebDriverContainer? I faced the same issue, and haven't found quick way (probably I misread some docs after all).

@rnorth
Copy link
Member

rnorth commented Sep 11, 2016

@dmart28 yes - probably would be a good idea to allow the selenium container image to be overridden (not just the version tag).

@magnusb
Copy link

magnusb commented Nov 26, 2016

Yes, the containers for 3.01 published by selenium seem to have a suffix so they can not be used at the moment. See the tags here: https://hub.docker.com/r/selenium/standalone-firefox/tags/

Being able to override the container template string would be very helpful.

rnorth added a commit that referenced this issue Dec 4, 2016
…ys trying to figure out from classpath dependencies. Refs #171
@bsideup bsideup added this to the 1.1.8 milestone Jan 15, 2017
@rnorth rnorth closed this as completed Jan 22, 2017
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

No branches or pull requests

5 participants