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

Firefox test related to launching the browser is failing on Mac #8283

Closed
OrKoN opened this issue Apr 27, 2022 · 11 comments
Closed

Firefox test related to launching the browser is failing on Mac #8283

OrKoN opened this issue Apr 27, 2022 · 11 comments
Assignees

Comments

@OrKoN
Copy link
Collaborator

OrKoN commented Apr 27, 2022

See https://github.com/puppeteer/puppeteer/runs/6189321000?check_suite_focus=true

@OrKoN
Copy link
Collaborator Author

OrKoN commented Apr 27, 2022

Henrik, do you know what might have broken there?

@whimboo
Copy link
Collaborator

whimboo commented Apr 27, 2022

Note that the test has been disabled via PR #8286.

@OrKoN do you have some more details? How often is this test failing? Is it permanent or intermittent? And it seems to be MacOS only, right?

The failure is actually caused by a timeout:
https://github.com/puppeteer/puppeteer/runs/6189321000?check_suite_focus=true

  1. Launcher specs
    Puppeteer
    Puppeteer.launch
    should have custom URL when launching browser:
    Error: Timeout of 25000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/runner/work/puppeteer/puppeteer/test/launcher.spec.ts)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)

Maybe it's related to my fix on bug https://bugzilla.mozilla.org/show_bug.cgi?id=1759169 and that we face a shutdown hang of Firefox here.

@OrKoN
Copy link
Collaborator Author

OrKoN commented Apr 27, 2022

I saw it fail consistently on multiple CLs and also locally on my Mac. Locally, I tried to debug it and the browser was started successfully but the call to get pages was timing out. It looks like page initialisation was locking on some events.

@whimboo
Copy link
Collaborator

whimboo commented May 2, 2022

@OrKoN could you do me a favor please? Could you run again locally with DUMPIO=1 and by having the Firefox preference remote.log.level set to Trace (to be added here)? I would be kinda interested to see the complete log output for this test. Thanks!

@OrKoN
Copy link
Collaborator Author

OrKoN commented May 3, 2022

@whimboo sorry, I didn't get to this yet. I will try to take a look tomorrow.

@OrKoN
Copy link
Collaborator Author

OrKoN commented May 4, 2022

@stale
Copy link

stale bot commented Jul 3, 2022

We're marking this issue as unconfirmed because it has not had recent activity and we weren't able to confirm it yet. It will be closed if no further activity occurs within the next 30 days.

@stale stale bot added the unconfirmed label Jul 3, 2022
@whimboo
Copy link
Collaborator

whimboo commented Jul 4, 2022

@OrKoN I completely missed your reply, sorry. I had a look but it's not obvious what's Puppeteer is actually waiting for. The last initial command after the connection that gets successfully run is Network.enable. What else runs next eg. are there some events that Puppeteer is waiting for?

I wonder if this is still an issue for you or if it has been gone given that 2 months have been passed by.

@stale stale bot removed the unconfirmed label Jul 4, 2022
@stale
Copy link

stale bot commented Sep 2, 2022

We're marking this issue as unconfirmed because it has not had recent activity and we weren't able to confirm it yet. It will be closed if no further activity occurs within the next 30 days.

@stale stale bot added the unconfirmed label Sep 2, 2022
@whimboo
Copy link
Collaborator

whimboo commented Sep 2, 2022

@OrKoN any feedback from your side? Is it still valid?

@stale stale bot removed the unconfirmed label Sep 2, 2022
@OrKoN
Copy link
Collaborator Author

OrKoN commented Sep 2, 2022

Let's close this. There are still some flaky failures but we currently don't have Mac failures to block CI (only firefox)

@OrKoN OrKoN closed this as completed Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants