docs: update deimos ibc channel for osmosis #3954
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
TIL about #3906. We haven't shipped that for 68 yet, but probably should. Regardless, this docs PR should be updated to use that new functionality, because that's the current state of play. |
conorsch
force-pushed
the
docs-osmosis-channel
branch
from
bd45244 to
4263fc0
Compare
conorsch
force-pushed
the
docs-osmosis-channel
branch
from
4263fc0 to
1770c5a
Compare
We had a few chain halts on the `v0.68.x` release series. As of `v0.69.0`, "channel-0" is the correct value for relaying to Osmosis testnet. Added mention of the slick new `pcli query ibc channels` to the docs. Also overhauled the dev docs for how to test IBC functionality interactively, since that knowledge has mostly been communicated via pairing session, but not yet fully written down. We no longer use `rly`, neither for running relayer instances nor for ad-hoc CLI transfers. Instead, we can use `hermes` for both roles. Linked out to wiki docs on the high-churn hermes relayer config steps, since we're updating those frequently.
conorsch
force-pushed
the
docs-osmosis-channel
branch
from
1770c5a to
6453242
Compare
|
This grew a bit but is now an accurate and rather comprehensive description of how we're interacting with the IBC functionality to evaluate behavior. Not included is mention of Noble interactive testing steps, but mention of the Noble Hermes config is there. |
cratelyn
added
A-IBC
cratelyn
approved these changes
Mar 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We had a few chain halts on the
v0.68.xrelease series. As ofv0.69.0, "channel-0" is the correct value for relaying to Osmosis testnet. Added mention of the slick newpcli query ibc channelsto the docs.Also overhauled the dev docs for how to test IBC functionality interactively, since that knowledge has mostly been communicated via pairing session, but not yet fully written down. We no longer use
rly, neither for running relayer instances nor for ad-hoc CLI transfers. Instead, we can usehermesfor both roles.Linked out to wiki docs on the high-churn hermes relayer config steps, since we're updating those frequently. Therefore review should also include eyes on https://github.com/penumbra-zone/penumbra/wiki/Updating-Hermes