[BUG] ECONNRESET in smoke tests #6793
Labels
Comments
lukekarrys
added
Bug
lukekarrys
added a commit
that referenced
this issue
Sep 15, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
lukekarrys
added a commit
that referenced
this issue
Sep 16, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
lukekarrys
added a commit
that referenced
this issue
Sep 16, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
lukekarrys
added a commit
that referenced
this issue
Sep 20, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
lukekarrys
added a commit
that referenced
this issue
Sep 20, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
wraithgar
pushed a commit
that referenced
this issue
Sep 20, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
lukekarrys
added
Priority 2
lukekarrys
added a commit
that referenced
this issue
Oct 6, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
lukekarrys
added a commit
that referenced
this issue
Oct 6, 2023
The proxy package is more spec compliant and works out-of-the-box so we no longer have to pass --registry=PROXY to the tests. The goal here is to make it simpler to help debug #6793.
|
This was fixed by fixing the max sockets behavior in our http agent library. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current Behavior
The smoke test added in #6781 is failing (flaky) in Node 18 on macOS with
ECONNRESETerrors. This could be due to the new@npmcli/agent. I have not been able to reproduce this issue not in the smoke tests so I'm currently not sure of the severity of the issue.The main difference in smoke tests is that they are calling
npmvia@npmcli/promise-spawninstead of directly. This is just a note and I'm not sure yet of its relevance.Example of a flaky test run: https://github.com/npm/cli/actions/runs/6149382937/job/16685141474
Expected Behavior
The smoke tests pass without flakiness.
The text was updated successfully, but these errors were encountered: