-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
Electron 8: Linux unit tests are not starting #95415
Comments
Pushed changes that seem to help macOS, but Linux is still entirely broken and not even starting the tests:
|
@bpasero this is the reason why the gpu process is crashing
Can we free up some space in the vm ? |
@deepak1556 interesting, to be clear, is this disk space to free or more memory? |
On linux |
I don't think that command line argument helps, I can reproduce now running the docker image:
Any clues how to make changes to the shared memory thingy to try? |
The command line did work, gpu process no longer crashes. The fatal error that is remaining in the output is wrt initializing headless display. Some dependency changed in 8 that our docker container is missing, will look into it tomorrow. |
Okay its an upstream bug fixed by https://bugs.chromium.org/p/chromium/issues/detail?id=1025266#c19 , will backport the fix in electron. |
Oh ok great, let me know when I can try again. |
This seems fine now. Though I am seeing more test failures now in the integration tests. |
All good now. |
@bpasero I was trying to find the build/change that fixed this? Did the fix get backported to electron? |
@jeffrafter we didn't backport the chromium fix, the right solution here is to avoid using 8bit color depth with xvfb that is no longer supported by chromium. Can use 16, 24 or 32 and pass it to xvfb options like |
It seems evident that we have issues running our tests with Electron 8.
E.g. https://monacotools.visualstudio.com/DefaultCollection/Monaco/_build/results?buildId=71995&view=results
The text was updated successfully, but these errors were encountered: