test: fix test-cluster-send-handle-large-payload#19311
test: fix test-cluster-send-handle-large-payload#19311Trott wants to merge 1 commit intonodejs:masterfrom
Conversation
test-cluster-send-handle-large-payload is susceptible to failure if it is competing for resources. Move to `sequential` directory so it is not competing with other tests. Fixes: nodejs#14844
|
This will make the test less flaky but the |
@santigimeno Stress tests on FreeBSD show that there the test does not fail except when run in parallel (that is, with |
|
Yeah, I don't disagree. I meant that the underlying issue will still be there though less visible. |
Oh, I see. Yes. If we can find a way to trigger the issue reliably without consuming a ton of resources, we could create a |
|
Landed in 5e23b65 |
test-cluster-send-handle-large-payload is susceptible to failure if it is competing for resources. Move to `sequential` directory so it is not competing with other tests. Fixes: nodejs#14844 PR-URL: nodejs#19311 Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
test-cluster-send-handle-large-payload is susceptible to failure if it
is competing for resources. Move to
sequentialdirectory so it is notcompeting with other tests.
Fixes: #14844
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes