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

Web Tests are constantly failing for me under MacOS in CI pipeline #43

Closed
dbaeumer opened this issue Sep 23, 2022 · 3 comments
Closed

Comments

@dbaeumer
Copy link
Member

A typical failure can be seen here:

https://dev.azure.com/vscode/vscode-wasm/_build/results?buildId=74201&view=logs&j=a5e52b91-c83f-5429-4a68-c246fc63a4f7&t=8b9bff53-2841-58c7-68fc-e30c30330fc3

> node lib/web/runTests.js

node:events:504
      throw er; // Unhandled 'error' event
      ^

Error: read ECONNRESET
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:217:20)
Emitted 'error' event on ClientRequest instance at:
    at TLSSocket.socketErrorListener (node:_http_client:442:9)
    at TLSSocket.emit (node:events:526:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -54,
  code: 'ECONNRESET',
  syscall: 'read'
}
@aeschli
Copy link
Contributor

aeschli commented Oct 7, 2022

I got the same issue after we moved the CI build from gitHub Actions to dev azure.

I now added a uncaughtException handler to the test runner (when running test-web from the CLI). This seem to help:
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=188358&view=logs&j=4bc49d80-bf2e-5a4d-f40c-10b2635a6a32&t=bece58dd-0f02-58e2-58d7-904a157f0ec5&l=61
The exception is logged and ignore, and didn't seem to cause any trouble.

I think you use the web-test API. So you want to add a uncaughtException to your test runner as well.

Where the exception is coming from is hard to say:
nodejs/node#27916
webpack/webpack#15501
mark-wiemer/hello-vscode-extension#10

@aeschli
Copy link
Contributor

aeschli commented Oct 7, 2022

nodejs/node#35824

@dbaeumer
Copy link
Member Author

Added the handler.

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