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

Fix issue that prevented Mocha and generator tests from running on certain configurations #5344

Merged
merged 2 commits into from
Aug 10, 2021

Conversation

gonfunko
Copy link
Contributor

@gonfunko gonfunko commented Aug 10, 2021

This PR adds the --disable-gpu flag to the run mocha/generator test scripts, which works around an issue where Chrome would hang when running the tests on Linux with older versions of the NVIDIA driver.

This fixes #5345.

@gonfunko gonfunko requested a review from a team as a code owner August 10, 2021 21:09
@gonfunko gonfunko requested a review from maribethb August 10, 2021 21:09
@google-cla google-cla bot added the cla: yes Used by Google's CLA checker. label Aug 10, 2021
@@ -51,8 +51,10 @@ async function runGeneratorsInBrowser() {
args: ['--headless', '--no-sandbox', '--disable-dev-shm-usage', '--allow-file-access-from-files']
};
} else {
// --disable-gpu is needed to prevent Chrome from hanging on Linux with
// NVIDIA drivers older than v295.20.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add a link here and below to #5345 so we have the future context, please?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@gonfunko gonfunko merged commit a04debf into google:goog_module Aug 10, 2021
@gonfunko gonfunko deleted the gpu_flag branch August 10, 2021 22:11
@github-actions github-actions bot added this to the 2021_q3_release milestone Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Used by Google's CLA checker. type: cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants