-
Notifications
You must be signed in to change notification settings - Fork 3.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
[BUG] browserType.launch: Protocol error (Browser.enable): Browser closed #11130
Comments
@bpasero I see this is Firefox crashing for you. A few questions:
|
Not on every run, but maybe once per week we get this
Yup we do. We start/stop tracing per unit test and only persist it on failure. Would it make sense to always set |
I can confirm that I have the same issue and it happens just sometimes randomly, and only on firefox (we run chromium, webkit and ff) |
It just happened again, we will probably have to disable Firefox to get non-flaky test runs. |
@bpasero we recently fixed a significant bug related to tracing in firefox that resulted in 72% of all the crashes we see. Can you please try running with @playwright/test@next? |
I would rather wait for a new official release from |
@bpasero scheduled for jan 2022! |
@bpasero we no longer think that we fixed it. Can you help us reproduce it? Maybe we can SSH into your hosted macOS runner and try reproduce it there? |
I do not think that is an option, after all this is our main Azure pipeline for building and we do not offer SSH access to that as far as I can tell for security reasons. If there is some debug flags I can enable, please let me know. Btw not sure if related, but in another run (not Firefox), I just got this now:
|
@bpasero can you please setup a |
|
Oh, maybe that is not a Firefox run actually, looks like But I really like this output, is there any harm just always running with these debug environment variable for all our runs across all platforms? |
@bpasero No harm, but it's spammy :) I see all the browsers are launched in parallel which makes them fight for system resources. This might be fine on a very powerful hardware, but in general this is slow and unrecommended. Any chance you can try running these tests serially instead? |
No that is not an option for us, because our builds are already very slow (more than 1h) and running tests in parallel was an explicit decision to speed up builds. We can always try to increase how beefy these machines are? |
@bpasero yes, at least 2 CPU cores per browser. Or can you triple the machine count - and run one per browser? This is what we ourselves do. |
After discussion with team we have decided to stop running builds in parallel in our CI machines. Will report back how that goes over coming days. |
This is happening to me as well, but I'm using |
@iOnline247 this error might happen in all kinds of situations. Would you mind filing a separate issue with a description of what's going on? |
Some new errors we see even after running sequentially:
|
@bpasero and this now looks like something that was fixed! Let's see how it behaves with 1.18 |
Nice, is that version planned to be released soon? |
@bpasero 1.18.0-rc1 was just released; would you mind giving it a try?
|
Ok will try. |
I am getting weird compile errors now:
Did anything change besides the package having moved to You should be able to reproduce that from https://github.com/microsoft/vscode/tree/ben/playwright-1.18 with the command: |
Nevermind my above comment, I figured out a bad old TS dependency on our end. |
Pushed to |
Just came across this one:
|
@bpasero so I tried running firefox multiple times, though I can't repro this. How often does the error happen these days? Also, can you please share the parameters of your mac runner? Maybe I can get a similar setup and see if things will repro. |
We no longer run Firefox tests so I cannot answer that question. Our Azure pipeline configuration is https://github.com/microsoft/vscode/blob/main/build/azure-pipelines/darwin/product-build-darwin.yml |
I so far still failed to repro this locally, but I'll try more! @bpasero the azure pipeline you referred to has lots of secrets that are not accessible to me. Any instructions how to reproduce this locally as close as possible to your CI setup? |
Is there any workaround to run tests on Firefox and Webkit? unable to launch on windows 10.... and npx playwright install is giving perm error. browserType.launch: Executable doesn't exist at C:<>\AppData\Local\ms-playwright\firefox-1313\firefox\firefox.exe ╔═════════════════════════════════════════════════════════════════════════╗ npx playwright install |
I pinned the version to 1.15.0 and it went away, FWIW. |
@bpasero: I'm happy to come back to this is there's a good repro that we can easily run & debug locally. Otherwise, we're out ideas! Everybody: most of the other comments here seem to be unrelated to the bug discussed. Please, file separate issues! |
@mxschmitt , It worked ! , but now I try to do it with dockers and it gives me an error, has it happened to you?¡hola! Trato de levantar en un servidor ubuntu, pero me da este error :(((( |
Lets file a new issue please, since this one is unrelated and continues to spam others. |
Context:
Code Snippet
Describe the bug
We had a failing Azure pipelines CI run in https://dev.azure.com/monacotools/Monaco/_build/results?buildId=148061&view=logs&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=0d7e5bc9-922f-51dd-b06a-a90d0b9feeeb&l=13 with the following output printed:
Asking here for advice if there is something we can do about it.
The text was updated successfully, but these errors were encountered: