confio / ts-relayer

IBC Relayer in TypeScript
MIT License
109 stars 63 forks source link

Make poll time configurable #239

Closed ethanfrey closed 2 years ago

ethanfrey commented 2 years ago

We were hardcoding some sleeps that made sense for CI. These may be creating the problems in running on mainnets. Making them configurable in the registries and using conservative values.

changeset-bot[bot] commented 2 years ago

🦋 Changeset detected

Latest commit: 389f95c03d182e02362ede58c4f41b4f43ab5b38

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | --------------- | ----- | | @confio/relayer | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

ethanfrey commented 2 years ago

@daniel-farina happy if you can check this out (try the code) - you will need to update your registry.yaml file

sascha1337 commented 2 years ago

can we get ordered timeouts, to use ICA with ts-relayer ? like here? https://github.com/cosmos/relayer/commit/8d5f79207ae9d1dcc8cb3f8210e0f05cb6c39688#diff-ed9e8e86a269a3118fe1634dc1559607ffa6a11e3b4fd6457122b2888c4f4056

ethanfrey commented 2 years ago

This is a long Wishlist https://github.com/confio/ts-relayer/issues/47

Not sure when I do it, but happy for a PR. Let's see how much uptake comes with ts-relayer.