Investigate why sequential/test-gc-http-client-onerror is flaky

[mcollina]

13:14:19 not ok 3429 sequential/test-gc-http-client-onerror
13:14:21 ---
13:14:21 duration_ms: 1.623
13:14:21 severity: fail
13:14:21 exitcode: 1
13:14:21 stack: |-
13:14:21 done/collected/total: 160/0/640
13:14:21 done/collected/total: 314/160/640
13:14:21 done/collected/total: 480/256/640
13:14:21 done/collected/total: 630/314/640
13:14:21 done/collected/total: 640/480/640
13:14:21 done/collected/total: 640/630/640
13:14:21 done/collected/total: 640/640/640
13:14:21 node:_http_server:493
13:14:21 if (connections[i].socket._httpMessage && !connections[i].socket._httpMessage.finished){
13:14:21 ^
13:14:21 
13:14:21 TypeError: Cannot read properties of null (reading '_httpMessage')
13:14:21 at Server.closeIdleConnections (node:_http_server:493:31)
13:14:21 at Server.close (node:_http_server:476:8)
13:14:21 at Immediate.status (/home/iojs/build/workspace/node-test-binary-armv7l/test/sequential/test-gc-http-client-onerror.js:65:14)
13:14:21 at process.processImmediate (node:internal/timers:471:21)
13:14:21 
13:14:21 Node.js v19.0.0-pre

[tniessen]

Related: #43638

Marking as flaky: #43957

[F3n67u]

#43949 is trying to fix this problem. Could we favor #43949 over mark it as flaky? @tniessen

[tniessen]

If it does indeed fix the problem, I'd favor the fix over marking it as flaky, of course. I don't have enough time nor enough insight into HTTP internals to say if it does fix the problem, but we do need to resolve all of the issues caused by #43522 quickly.

[F3n67u]

If it does indeed fix the problem, I'd favor the fix over marking it as flaky, of course. I don't have enough time nor enough insight into HTTP internals to say if it does fix the problem, but we do need to resolve all of the issues caused by #43522 quickly.

I agree with you. We could mark it flaky first and remove it from the flaky list when we are sure that #43949 does address this flaky test.

[F3n67u]

fixed by #43949