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

Add --disable-gpu in the default Chromium args #262

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

AgnesToulet
Copy link
Contributor

While doing some load testing of the application with the default rendering mode, I noticed high CPU and memory slowy going to the roof after the tests were finished. I narrowed down the issue to the newPage function that was (for some requests) hanging forever, preventing the browser to be closed and causing these issues.

It seems like adding the --disable-gpu flag to the Chromium args fixed the issue.

Should fix #229

@AgnesToulet AgnesToulet added type/bug Something isn't working component/chromium labels Jul 30, 2021
Copy link
Member

@marefr marefr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM I guess, but wonder what possible issues this might cause for others basically switching from hardware acceleration to software rendering if I understand it correctly? Guess just try, but make sure to make the change clear in the change log should anyone have issues with it.

@AgnesToulet AgnesToulet merged commit 035e3d0 into master Aug 25, 2021
@AgnesToulet AgnesToulet deleted the update-default-flag branch August 25, 2021 07:17
@jgournet
Copy link

thank you for this !
quick question: is the build & push to docker hub automated, or does someone need to trigger it ? (just so the image gets this change)

@AgnesToulet
Copy link
Contributor Author

It's updated only with a new release. Next release should happen this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/chromium type/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Browser page crashed
3 participants