nodejs / node

Node.js JavaScript runtime ✨🐢🚀✨
https://nodejs.org
Other
107.7k stars 29.64k forks source link

parallel/test-http2-large-write-multiple-requests is flaky #54885

Closed lpinca closed 2 months ago

lpinca commented 2 months ago

Test

test-http2-large-write-multiple-requests

Platform

Linux x64

Console output

$ ./tools/test.py --repeat=10000 test/parallel/test-http2-large-write-multiple-requests.js
=== release test-http2-large-write-multiple-requests ===
Path: parallel/test-http2-large-write-multiple-requests

...

Command: out/Release/node /home/luigi/node/test/parallel/test-http2-large-write-multiple-requests.js
--- TIMEOUT ---

...

[05:05|%  99|+ 9996|-   3]: Done

Failed tests:
out/Release/node /home/luigi/node/test/parallel/test-http2-large-write-multiple-requests.js
out/Release/node /home/luigi/node/test/parallel/test-http2-large-write-multiple-requests.js
out/Release/node /home/luigi/node/test/parallel/test-http2-large-write-multiple-requests.js

Build links

Additional information

Similarly to parallel/test-net-write-fully-async-hex-string, the test correctly finished but the process does not exit. This

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.

lpinca commented 2 months ago

Duplicate of https://github.com/nodejs/node/issues/52550.