-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
flaky: parallel/test-http2-large-write-multiple-requests #52550
Comments
hmm, maybe we can add an unref'ed timer in the test that logs out the active handles after around 1 minute to figure out what's keeping the process alive.. |
I'm unable to reproduce locally on macOS: $ ./tools/test.py --repeat 9999 -t 2 test/parallel/test-http2-large-write-multiple-requests.js
[05:07|% 100|+ 9999|- 0]: Done
All tests passed. |
I can reproduce on Ubuntu
Similarly to diff --git a/test/parallel/test-http2-large-write-multiple-requests.js b/test/parallel/test-http2-large-write-multiple-requests.js
index bcbb1434cb..b5a698e47f 100644
--- a/test/parallel/test-http2-large-write-multiple-requests.js
+++ b/test/parallel/test-http2-large-write-multiple-requests.js
@@ -1,4 +1,5 @@
'use strict';
+// Flags: --jitless
const common = require('../common');
if (!common.hasCrypto)
common.skip('missing crypto');
fixes the issue for both tests on my machine. |
I'm able to relatively quickly reproduce on https://ci.nodejs.org/computer/test%2Dibm%2Drhel8%2Dx64%2D3/ After attaching gdb to the hanging process, I get this:
|
In other words, a revert of e600de9 might make the issues much easier to reproduce on fast CPUs |
FWIW, I get no failures (for this test) with the patch from #47452 applied on main. |
I confirm. Let's reopen that pull request. |
The are a bunch of other tests that fail with that patch, though. |
Test
parallel/test-http2-large-write-multiple-requests
Platform
Linux x64
Console output
Build links
Additional information
No response
The text was updated successfully, but these errors were encountered: