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.

Sign up for GitHub

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

Jump to bottom

Relay on all paths #867

Closed
5 tasks
romac opened this issue Apr 28, 2021 · 1 comment
Closed
5 tasks

Relay on all paths #867

romac opened this issue Apr 28, 2021 · 1 comment
Assignees
@romac
Labels
E: gravity External: related to Gravity DEX I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support
Milestone
05.2021

Comments

@romac
Copy link
Member

romac commented Apr 28, 2021 •
edited
Loading

Crate

relayer, relayer-cli

Summary

The relayer should relay on all existing paths between the chains in the config, not just the ones defined in the [connections] section (which will be removed). To filter out which paths to relay on, a policy language will be introduced at a later stage.

Problem Definition

This is a hard requirement for the Gravity DEX.

Proposal

Listen from events from all configured chains and start workers as needed.

Acceptance Criteria

When the start-multi command relays all incoming packets from all the chains in the config.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@romac romac added O: new-feature Objective: cause to add a new feature or support in progress I: logic Internal: related to the relaying logic labels Apr 28, 2021
@romac romac self-assigned this Apr 28, 2021
@romac romac added this to the 05.2021 milestone Apr 28, 2021
@romac romac mentioned this issue Apr 28, 2021
Closed
11 tasks
@romac romac removed the in progress label Apr 28, 2021
@adizere adizere mentioned this issue Apr 29, 2021
Closed
5 tasks
@romac romac changed the title Discover and relay on all paths if [paths] section is omitted in the config Relay on all paths Apr 29, 2021
@andynog andynog added the E: gravity External: related to Gravity DEX label Apr 29, 2021
@ancazamfir
Copy link
Collaborator

Covered by #862, #784

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@romac romac

Labels
E: gravity External: related to Gravity DEX I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support
Projects
None yet
Milestone
05.2021
Development

No branches or pull requests
3 participants
@romac @ancazamfir @andynog