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

Disable Chrome's GPU support in BrowserWebDriverContainer #4315

Merged
merged 7 commits into from
Jul 30, 2021
Merged

Conversation

kiview
Copy link
Member

@kiview kiview commented Jul 30, 2021

Yesterday's release of docker-selenium 3.141.59-20210729 seems to fail to start under different circumstances and in different environments.

A potential fix is setting --disable-gpu in the ChromeOptions.

It is not yet clear what is the root cause of this issue (the update to Chrome 92?), but other users of docker-selenium report issues as well:
SeleniumHQ/docker-selenium#1346

This PR shall act as a way to further debug this issue in GitHub Actions and to discuss potential workarounds on our side.

@kiview kiview changed the title Investigate issues with selenium/standalone-chrome-debug:3.141.59 in BaseWebDriverContainerTest Investigate issues with selenium/standalone-chrome-debug:3.141.59 in BrowserWebDriverContainer Jul 30, 2021
@kiview kiview marked this pull request as ready for review July 30, 2021 14:50
@kiview kiview requested review from bsideup and rnorth as code owners July 30, 2021 14:50
@kiview
Copy link
Member Author

kiview commented Jul 30, 2021

I added a very specific hack for the current Selenium web driver version in combination with Chrome into BrowserWebDriverContainer. Even having a single container launched without this flag can have catastrophic consequences for the whole pipeline (as seen in the pipelines of the different commits in this PR), possibly from CPU resource exhaustion.

Of course, we can try to be even more specific in checking for the need for this hack, but so far, this is my first try, considering that Selenium have updated their existing tags.

@kiview kiview changed the title Investigate issues with selenium/standalone-chrome-debug:3.141.59 in BrowserWebDriverContainer Disable GPU for selenium/standalone-chrome-debug:3.141.59 in BrowserWebDriverContainer Jul 30, 2021
@bsideup bsideup added this to the next milestone Jul 30, 2021
@bsideup bsideup changed the title Disable GPU for selenium/standalone-chrome-debug:3.141.59 in BrowserWebDriverContainer Disable Chrome's GPU support in BrowserWebDriverContainer Jul 30, 2021
@bsideup bsideup merged commit 520cfeb into master Jul 30, 2021
@delete-merged-branch delete-merged-branch bot deleted the chrome-debug branch July 30, 2021 21:08
@kiview
Copy link
Member Author

kiview commented Aug 2, 2021

For documentation's sake, this seems to be the corresponding Chromium issue.

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.

2 participants