-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
Intermittent timeouts in test-tls-min-max-version.js #28337
Comments
Can you post the output? Also, just to confirm, what were your configure options? |
I ran ./configure with no options; where should I look to find any info on what it determined that will be helpful? I unfortunately did not save the output and am not currently able to reproduce. :( I'll keep trying and post it if I repro it again. Sorry for not thinking of that. |
Knowing that there were no options is enough. Without some output, its hard to guess, much less repro. This runs regularly in CI on macos, and many (most?) devs are using macos, so its surprising. It also is a pretty simple test with no subtle timing races or ordering dependencies. Which doesn't mean it doesn't have a bug, but does mean that some clues are needed. I don't even know what timeout you hit, if you run it standalone, I didn't think there were any timeouts. |
I can reproduce with
on master. macOS 10.14.5 |
I dug a little and noticed a bunch of node/test/fixtures/tls-connect.js Line 70 in 35cb955
is emitted but node/test/fixtures/tls-connect.js Line 81 in 35cb955
is not. As a result the callback is not called and test times out. I've increased the ephemeral port range
and reduced the duration of the
and now it works with no timeouts. I'm not sure if there is anything actionable to do. |
Closing as there was no recent activity |
Attempting to run
make test
, or runningtest/parallel/test-tls-min-max-version.js
in isolation I get intermittent timeouts.The text was updated successfully, but these errors were encountered: