hermes icon indicating copy to clipboard operation
hermes copied to clipboard

Dymension <-> RollupEVM relaying

Open ljoss17 opened this issue 11 months ago • 0 comments

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).

ljoss17 avatar Mar 15 '24 15:03 ljoss17