Skip to content

Commit 70bf51b

Browse files
sirtimidclaude
andcommitted
fix(kernel-node-runtime): increase ACK timeout for multi-peer reconnection test
The "handles multiple simultaneous reconnections to different peers" test has kernel1 redeem URLs on two peers that restart sequentially through a relay. The default ackTimeoutMs of 2s gives only 8s for URL redemption, which is too tight when two kernels are reconnecting in CI. Use 5s (→ 20s redemption window) for the initiating kernel. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent da71ee1 commit 70bf51b

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

packages/kernel-node-runtime/test/e2e/remote-comms.test.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -651,10 +651,15 @@ describe.sequential('Remote Communications E2E', () => {
651651
});
652652
let kernel3: Kernel | undefined;
653653

654+
// Use a longer ACK timeout for the initiating kernel — it must redeem
655+
// URLs on two peers that restart sequentially through the relay, so the
656+
// default 2s × (3+1) = 8s redemption window is too tight for CI.
657+
const multiPeerBackoff = { ...testBackoffOptions, ackTimeoutMs: 5_000 };
658+
654659
try {
655660
await kernel1.initRemoteComms({
656661
relays: testRelays,
657-
...testBackoffOptions,
662+
...multiPeerBackoff,
658663
});
659664
await kernel2.initRemoteComms({
660665
relays: testRelays,
@@ -757,7 +762,7 @@ describe.sequential('Remote Communications E2E', () => {
757762
}
758763
}
759764
},
760-
NETWORK_TIMEOUT * 3,
765+
NETWORK_TIMEOUT * 4,
761766
);
762767
});
763768

0 commit comments

Comments
 (0)