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

[Bug?]: Timeouts when using PnP with ESM on Node <=16.14 #4157

Closed
1 task
franciscofabian opened this issue Feb 24, 2022 · 10 comments
Closed
1 task

[Bug?]: Timeouts when using PnP with ESM on Node <=16.14 #4157

franciscofabian opened this issue Feb 24, 2022 · 10 comments
Labels
bug Something isn't working

Comments

@franciscofabian
Copy link

Self-service

  • I'd be willing to implement a fix

Describe the bug

Running jest tests in CI always timeout (10 minutes) with yarn 3.2.0, running with 3.1.1 and earlier versions it finishes in less than 10 seconds.

To reproduce

yarn jest

Environment

CircleCI 2.1

Additional context

No response

@franciscofabian franciscofabian added the bug Something isn't working label Feb 24, 2022
@RDIL
Copy link
Member

RDIL commented Feb 25, 2022

Can you please provide a repository which exhibits the behavior?

@franciscofabian
Copy link
Author

I have fixed the issue adding the option --maxWorkers=1 in jest. I tried to set up a new repository to reproduce the behavior but with the same code an setup and couldn't reproduce, I'm not sure why yet. Is there any change in yarn 3.2.0 that could had caused the process not to exit when running in multiple workers?

@psychobolt
Copy link

I think I am also seeing the same issue on the rollup-terser-plugin, which utilizes jest workers as well... hmm

@arcanis
Copy link
Member

arcanis commented Feb 27, 2022

Does it work if you either disable esm support in your yarnrc, or upgrade to the very latest Node 16.x release?

@franciscofabian
Copy link
Author

Yes! It works for me either disabling esm support in my yarmrc or upgrading to the latest Node (v16.14.0 from v16.2.0)

@arcanis
Copy link
Member

arcanis commented Feb 27, 2022

Then it's caused by nodejs/node#41134

More details in this other thread: vercel/next.js#34688 (comment) (not related to Next.js, just discussion occurred there).

@merceyz
Copy link
Member

merceyz commented Apr 13, 2022

To have it written in this issue as well:
This issue was fixed in nodejs/node#41221 which has shipped in Node.js v14.19.2, v16.14, and v17.4.

@sublimator
Copy link

sublimator commented Apr 22, 2022

This issue was fixed in nodejs/node#41221

Will this be fixed in 14 LTS ?

@merceyz
Copy link
Member

merceyz commented Apr 22, 2022

I don't know, this is the wrong place to ask.

@sublimator
Copy link

Fair enough, thanks

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

No branches or pull requests

6 participants