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

Epic: basic self relaying via the CLI / explorer / warp UI #3448

Open
5 of 7 tasks
Tracked by #166
tkporter opened this issue Mar 20, 2024 · 3 comments
Open
5 of 7 tasks
Tracked by #166

Epic: basic self relaying via the CLI / explorer / warp UI #3448

tkporter opened this issue Mar 20, 2024 · 3 comments
Assignees

Comments

@tkporter
Copy link
Collaborator

tkporter commented Mar 20, 2024

High level outcomes

  • Only MessageIdMultisig ISMs, Routing ISMs, Aggregation ISMs, and Null ISM for now
  • Able to self relay using the warp route UI, explorer, and CLI
  • Everything is built in a protocol-agnostic manner, but we just implement the "view calls" etc that are involved for EVM for now

Motivation:

  • We often just use combinations of aggregation, routing, and multisig ISMs
  • We just focus on the MessageIdMultisig ISM type, as creating a merkle proof for a MerkleRootMultisig is heavyweight and atm only reasonable to do in the relayer. As it stands now, it would likely require indexing all messages, which is unreasonable to do in the CLI or explorer
  • We want to lower the time to success in testing that e.g. a warp route works
  • Allowing self-relaying if the relayer is having issues is a nice option for users
  • We're not going with a shared WASM implementation between Rust and Typescript for a few reasons:

See Notion for the full thought process around this

OPTIONAL & dependent on prioritization

Preview Give feedback
  1. sdk

Metadata building in the SDK

Preview Give feedback
  1. sdk
    yorhodes
  2. sdk
    yorhodes
  3. sdk
    yorhodes

Self-delivery of messages

Preview Give feedback
  1. explorer
  2. warp-UI
@avious00 avious00 moved this to Icebox in Hyperlane Tasks Mar 20, 2024
@tkporter tkporter changed the title Epic: basic self relaying via the CLI & explorer Epic: basic self relaying via the CLI / explorer / warp ui Mar 20, 2024
@tkporter tkporter changed the title Epic: basic self relaying via the CLI / explorer / warp ui Epic: basic self relaying via the CLI / explorer / warp UI Mar 20, 2024
@avious00 avious00 added the agent label Mar 27, 2024
@avious00
Copy link
Contributor

hey @tkporter can you merge #2213 into this :)

@avious00 avious00 moved this from Icebox to Backlog in Hyperlane Tasks Mar 28, 2024
@avious00 avious00 added the permissionless permissionless deployments PI label Mar 28, 2024
@avious00
Copy link
Contributor

avious00 commented Mar 28, 2024

this will be helpful for developers to get up and running quickly and validate a testnet deployment

@tkporter
Copy link
Collaborator Author

hey @tkporter can you merge #2213 into this :)

sounds good, imo this epic is just a successor of that issue and there isn't anything else to roll in

@avious00 avious00 moved this from Backlog to Next Sprint in Hyperlane Tasks Apr 16, 2024
github-merge-queue bot pushed a commit that referenced this issue Jul 17, 2024
### Description

- implement `HyperlaneRelayer` as a wrapper around `HyperlaneCore`
- add relaying to CLI status command
- add infra script relay for an environment

### Drive-by changes

<!--
Are there any minor or drive-by changes also included?
-->

### Related issues

- Touches
#3448

### Backward compatibility

<!--
Are these changes backward compatible? Are there any infrastructure
implications, e.g. changes that would prohibit deploying older commits
using this infra tooling?

Yes/No
-->

### Testing

Manual

---------

Co-authored-by: -f <kunalarora1729@gmail.com>
Co-authored-by: Noah Bayindirli 🥂 <noah@primeprotocol.xyz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

3 participants