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

relayer: only configure penumbra chains #2804

Merged
merged 1 commit into from
Jul 7, 2023
Merged

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented Jul 7, 2023

We encountered a long (~20s) timeout on all rly commands, and traced it to a context timeout on a bad URL for the bablyon testchain. We're still focusing on preview <-> devnet functionality, so let's exclude the external testnet chains from relayer configuration until we're ready.

Also includes some documentation updates so it's a bit easier to understand how test the relayer interactively. Eventually we'll want all this logic bottled up in integration tests, e.g. interchaintest, but still working toward that goal.

Refs #2612.

We encountered a long (~20s) timeout on all `rly` commands, and traced
it to a context timeout on a bad URL for the bablyon testchain. We're
still focusing on preview <-> devnet functionality, so let's exclude the
external testnet chains from relayer configuration until we're ready.

Also includes some documentation updates so it's a bit easier to
understand how test the relayer interactively. Eventually we'll want all
this logic bottled up in integration tests, e.g. `interchaintest`, but
still working toward that goal.

Refs #2612.
@conorsch conorsch temporarily deployed to smoke-test July 7, 2023 22:17 — with GitHub Actions Inactive
@conorsch conorsch merged commit 8806a33 into main Jul 7, 2023
@conorsch conorsch deleted the relayer-qol-2023-07 branch July 7, 2023 22:40
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.

1 participant