test: fix flaky test-inspector#9727
Uh oh!
There was an error while loading. Please reload this page.
Conversation
Trott added test 
Trott commented Nov 21, 2016
gibfahn left a comment
gibfahn left a comment There was a problem hiding this comment.
LGTM. Interesting that it's the opposite to what @santigimeno suggested in #5386 (comment).
Is there a reason you only ran the test 100 times? Don't we normally run 9999 times to check for flakyness?
Normally, yes, but ~93% failure rate makes it seem like 100 is enough. But hey, let's go big: https://ci.nodejs.org/job/node-stress-single-test/1068/nodes=win10/console UPDATE: 9999 runs and it's all ✅ green. |
Trott commented Nov 21, 2016
CI is ✅ green! |
Using `socket.destroy()` instead of `socket.end()` fixes more-than-intermittent ECONNRESET issues on Windows. Fixes: nodejs#8804
Trottforce-pushed the fix-test-inspector branch from e1366de to dd78cc7CompareTrott commented Nov 22, 2016
Made a small change (removed the flaky designation for the test in |
Using `socket.destroy()` instead of `socket.end()` fixes more-than-intermittent ECONNRESET issues on Windows. PR-URL: nodejs#9727Fixes: nodejs#8804 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]>
Trott commented Nov 23, 2016
Landed in 2486273. Welcome back, green CI. Please stay a while, will ya? |
Using `socket.destroy()` instead of `socket.end()` fixes more-than-intermittent ECONNRESET issues on Windows. PR-URL: #9727Fixes: #8804 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]>
Using `socket.destroy()` instead of `socket.end()` fixes more-than-intermittent ECONNRESET issues on Windows. PR-URL: nodejs#9727Fixes: nodejs#8804 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]>
Using `socket.destroy()` instead of `socket.end()` fixes more-than-intermittent ECONNRESET issues on Windows. PR-URL: #9727Fixes: #8804 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]>
5 participants
Checklist
make -j8 test(UNIX), orvcbuild test nosign(Windows) passesAffected core subsystem(s)
test V8_inspector
Description of change
Using
socket.destroy()instead ofsocket.end()fixesmore-than-intermittent ECONNRESET issues on Windows.
Fixes: #8804
/cc @eugeneo@gibfahn
Stress test on master showing 93 failures in 100 runs on Windows 10: https://ci.nodejs.org/job/node-stress-single-test/1063/nodes=win10/console
Stress test with this change showing 0 failures in 100 runs on Windows 10:
https://ci.nodejs.org/job/node-stress-single-test/1066/nodes=win10/console