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

Relay for all paths defined in the configuration #748

Closed
5 tasks
romac opened this issue Mar 18, 2021 · 0 comments · Fixed by #742
Closed
5 tasks

Relay for all paths defined in the configuration #748

romac opened this issue Mar 18, 2021 · 0 comments · Fixed by #742
Assignees
Labels
I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support
Milestone

Comments

@romac
Copy link
Member

romac commented Mar 18, 2021

Crate

ibc-relayer, ibc-relayer-cli

Summary

The start-multi command should relay packets and channel events for all paths defined in the configuration.

Problem Definition

Despite its name, the start-multi command only relay packets and channel events for a single path between two chains.
We would instead like to have it relay between all paths defined in the configuration, as a first step towards a passive relayer.

Proposal

Initially, the start-multi command will spawn a supervisor per path defined in the configuration.

Specific care must be taken in the case when multiple paths exists, where the supervisor needs to filter out some events that may not be relevant to it.

For example, if two paths A(ibc0 -> ibc1) and B(ibc0 -> ibc2) exist, the supervisor for B must ensure that it discards the events emitted by ibc0 which are only relevant to worker B because they target the chain ibc2.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@romac romac self-assigned this Mar 18, 2021
@romac romac added this to the 03.2021 milestone Mar 18, 2021
@romac romac added O: new-feature Objective: cause to add a new feature or support I: logic Internal: related to the relaying logic labels Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant