From 9f658991eb38009ada8d1a6127ad1d6f323d326e Mon Sep 17 00:00:00 2001 From: Michael FIG Date: Sat, 2 May 2020 02:19:11 -0600 Subject: [PATCH] fix: decrease the need for sendPacket delay (almost fixed) --- .../cosmic-swingset/lib/ag-solo/vats/ibc.js | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/packages/cosmic-swingset/lib/ag-solo/vats/ibc.js b/packages/cosmic-swingset/lib/ag-solo/vats/ibc.js index c9120eb22b6..74152b5daf1 100644 --- a/packages/cosmic-swingset/lib/ag-solo/vats/ibc.js +++ b/packages/cosmic-swingset/lib/ag-solo/vats/ibc.js @@ -19,16 +19,15 @@ const DEFAULT_PACKET_TIMEOUT = 1000; // only way to create channels. const FIXME_ALLOW_NAIVE_RELAYS = true; -// FIXME: We need to delay to a later block to work around -// a relayer race condition (no sendPacket too soon after -// a channelOpenAck). +// We need to delay to a later block because packets +// cannot be sent before the channel ack is complete +// (no sendPacket within a channelOpenAck). // -// I[2020-05-01|15:32:43.721] - listening to tx events from ibc0... -// I[2020-05-01|15:32:43.721] - listening to block events from ibc0... -// Error: no transactions returned with query -// no transactions returned with query -// [exits] -const FIXME_SENDPACKET_DELAY_S = 10; // fail <=3, work >=4 +// FIXME: We might be fix within the relayer, which currently +// fails with: +// cannot update client with ID ibczeroclient: header blocktime +// ≤ latest client state block time (X ≤ X): invalid block header +const FIXME_SENDPACKET_DELAY_S = 1; /** * @typedef {import('@agoric/swingset-vat/src/vats/network').ProtocolHandler} ProtocolHandler