Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: override retry config, enforce large retry times, and fix min gas amount #12

Merged
merged 4 commits into from
Feb 3, 2023

Conversation

SebastianElvis
Copy link
Member

This PR fixes a number of issues reported during merging relayer to infra.
Specifically, this PR

  1. overrides retry config in the relayer dependency, such that all called functions in the relayer dependency apply our relayer config
  2. increases the retry time to 20 by default, and increases the retry interval to 1 second from 0.4 second
  3. sets min-gas-amount: 1 in example config

Rationale:

  • 12 aims at fixing the height requested is too high issue. The reason is that QueryLatestHeights returns the latest height that may not be committed yet. This makes any ABCI query with this height fails due to height requested is too high error. This is an known issue Light Client: "height requested is too high" thrown when querying against the latest height tendermint/tendermint#7641 and is fixed by a later PR to Tendermint. All we can do before bumping Tendermint is to retry more until the chain grows beyond that height.
  • 3 aims at fixing the Error building or broadcasting transaction issue. The reason is as described here.

@SebastianElvis SebastianElvis merged commit 642d835 into main Feb 3, 2023
@SebastianElvis SebastianElvis deleted the fix-retry branch February 3, 2023 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants