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

Investigate flaky test - test-gc-http-client-onerror.js #41399

Closed
mhdawson opened this issue Jan 4, 2022 · 1 comment · Fixed by #41619
Closed

Investigate flaky test - test-gc-http-client-onerror.js #41399

mhdawson opened this issue Jan 4, 2022 · 1 comment · Fixed by #41619
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. http Issues or PRs related to the http subsystem.

Comments

@mhdawson
Copy link
Member

mhdawson commented Jan 4, 2022

Test

test-gc-http-client-onerror.js

Platform

freebsd

Console output

/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd12-x64/test/parallel/test-gc-http-client-onerror.js:50
  throw err;
  ^

Error: connect ECONNRESET 127.0.0.1:10490 - Local (127.0.0.1:20028)
    at internalConnect (node:net:931:16)
    at defaultTriggerAsyncIdScope (node:internal/async_hooks:465:18)
    at GetAddrInfoReqWrap.emitLookup [as callback] (node:net:1075:9)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:74:8) {
  errno: -54,
  code: 'ECONNRESET',
  syscall: 'connect',
  address: '127.0.0.1',
  port: 10490
}

Node.js v18.0.0-pre

Build links

Additional information

No response

@mhdawson mhdawson added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Jan 4, 2022
@Mesteery Mesteery added the http Issues or PRs related to the http subsystem. label Jan 4, 2022
@lpinca
Copy link
Member

lpinca commented Jan 17, 2022

I can reproduce this by running

./tools/test.py --repeat=1000 test/parallel/test-gc-http-client-onerror

on macOS (16 parallel tasks) but I cannot reproduce it on Windows and Linux (8 parallel tasks).

I can make the test pass reliably on macOS by reducing parallelism, the duration of the TIME_WAIT state, and increasing the number ephemeral ports.

sudo sysctl -w net.inet.ip.portrange.first=32768
sudo sysctl net.inet.tcp.msl=1000
./tools/test.py -j5 --repeat=1000 test/parallel/test-gc-http-client-onerror

I wonder if it makes sense to move it to sequential like it was done for its companion test-gc-http-client.js in 7d03ff1.

lpinca added a commit to lpinca/node that referenced this issue Jan 20, 2022
nodejs-github-bot pushed a commit that referenced this issue Jan 22, 2022
Fixes: #41399

PR-URL: #41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
BethGriggs pushed a commit that referenced this issue Jan 25, 2022
Fixes: #41399

PR-URL: #41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Linkgoron pushed a commit to Linkgoron/node that referenced this issue Jan 31, 2022
Fixes: nodejs#41399

PR-URL: nodejs#41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
danielleadams pushed a commit that referenced this issue Feb 28, 2022
Fixes: #41399

PR-URL: #41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
danielleadams pushed a commit that referenced this issue Mar 2, 2022
Fixes: #41399

PR-URL: #41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
danielleadams pushed a commit that referenced this issue Mar 3, 2022
Fixes: #41399

PR-URL: #41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
danielleadams pushed a commit that referenced this issue Mar 14, 2022
Fixes: #41399

PR-URL: #41619
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI. http Issues or PRs related to the http subsystem.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants