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

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

Closed
lpinca opened this issue Sep 11, 2024 · 1 comment
Closed

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

lpinca opened this issue Sep 11, 2024 · 1 comment
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. flaky-test Issues and PRs related to the tests with unstable failures on the CI. linux Issues and PRs related to the Linux platform.

Comments

@lpinca
Copy link
Member

lpinca commented Sep 11, 2024

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 lpinca added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Sep 11, 2024
@github-actions github-actions bot added the linux Issues and PRs related to the Linux platform. label Sep 11, 2024
@lpinca
Copy link
Member Author

lpinca commented Sep 11, 2024

Duplicate of #52550.

@lpinca lpinca closed this as completed Sep 11, 2024
@avivkeller avivkeller added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. flaky-test Issues and PRs related to the tests with unstable failures on the CI. linux Issues and PRs related to the Linux platform.
Projects
None yet
Development

No branches or pull requests

2 participants