hermes
hermes copied to clipboard
Dymension <-> RollupEVM relaying
Closes: #XXX
Description
This PR explore the changes required to relay between Dymension and RollupEVM. In order to identify if what works and what requires changes, this PR adds changes needed to bootstrap both Dymension and RollupEVM in the integration tests.
The current findings:
- Changes to tendermint-rs are required in order to not force the Node ID to be of length 20. It seems that the Node ID is longer for RollupEVM, https://github.com/informalsystems/tendermint-rs/blob/v0.34.0/tendermint/src/node/id.rs?plain=1#L15.
- Since both Hermes and tendermint-rs depend on ibc-proto-rs, some update on ibc-proto-rs might be required to match versions used by both.
The current status for the transfer test:
- It seems that Send packet and Acknowledgment are seen and relayed
- Test fails due to receiving wallet not having the tokens when querying the balance on RollupEVM
PR author checklist:
- [ ] Added changelog entry, using
unclog
. - [ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
- [ ] Linked to GitHub issue.
- [ ] Updated code comments and documentation (e.g.,
docs/
). - [ ] Tagged one reviewer who will be the one responsible for shepherding this PR.
Reviewer checklist:
- [ ] Reviewed
Files changed
in the GitHub PR explorer. - [ ] Manually tested (in case integration/unit/mock tests are absent).