Skip to content

Commit ae8364c

Browse files
committed
CLDSRV-918: experiment - disable SDK request timeout
Comparison branch for the 10x flakiness measurement: identical to bugfix/CLDSRV-918-raise-test-request-timeout except requestTimeout is 0 instead of 30000, to compare stale-socket-timer reset rates.
1 parent 5a7ea24 commit ae8364c

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

  • tests/functional/aws-node-sdk/test/support

tests/functional/aws-node-sdk/test/support/config.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,12 @@ const DEFAULT_GLOBAL_OPTIONS = {
2626
// 120s; the v3 migration introduced a 5s timeout that heavy server-side
2727
// operations (multi-megabyte UploadPartCopy, 1000-key batch deletes) exceed
2828
// under CI load, as the server sends no response bytes until they complete.
29-
// 30s stays below mocha's 40s global timeout so a genuine hang still
30-
// surfaces as the SDK TimeoutError rather than a generic mocha timeout.
31-
const REQUEST_TIMEOUT = 30000;
29+
// 0 disables the SDK timeout entirely; mocha's 40s global timeout remains
30+
// the backstop for genuine hangs. Do not set a value of 6000 or more:
31+
// @smithy/node-http-handler then attaches the timer directly to pooled
32+
// keepAlive sockets without cleanup, and the stale timers destroy
33+
// connections out from under later requests.
34+
const REQUEST_TIMEOUT = 0;
3235

3336
const DEFAULT_MEM_OPTIONS = {
3437
endpoint: `${transport}://127.0.0.1:8000`,

0 commit comments

Comments
 (0)