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

Test ordered channel packet relaying correctness in the presence of concurrent relayers #2010

Open
5 tasks
soareschen opened this issue Mar 25, 2022 · 0 comments
Open
5 tasks

Test ordered channel packet relaying correctness in the presence of concurrent relayers #2010

soareschen opened this issue Mar 25, 2022 · 0 comments
Assignees
@soareschen
Labels
A: low-priority Admin: low priority / non urgent issue, expect longer wait time for PR reviews O: tests Objective: Test more aspect of the relayer
Milestone
Refactor sink

Comments

@soareschen
Copy link
Contributor

Summary

Write integration test to test whether the ordered channel packets are still relayed correctly when multiple relayers are racing to relay the packets.

The main complication in writing this test is that the two relayers would have to use different wallets in order to race relaying in parallel without account sequence mismatch errors. New methods will have to be added to RelayerDriver so that it can create a forked relayer with the config modified with new wallets for each chain.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@soareschen soareschen added the O: tests Objective: Test more aspect of the relayer label Mar 25, 2022
@soareschen soareschen changed the title Test ordered channel packet relaying correctness in the present of concurrent relayers Test ordered channel packet relaying correctness in the presence of concurrent relayers Mar 25, 2022
@soareschen soareschen mentioned this issue Mar 28, 2022
Closed
6 tasks
@adizere adizere added this to the v0.15.0 milestone Apr 25, 2022
@adizere adizere added the A: low-priority Admin: low priority / non urgent issue, expect longer wait time for PR reviews label Apr 25, 2022
@adizere adizere modified the milestones: v0.15.0, v1.0.0 May 2, 2022
@adizere adizere modified the milestones: v1.0.0, v1.2 May 24, 2022
@adizere adizere modified the milestones: v1.2, v1.1 Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@soareschen soareschen

Labels
A: low-priority Admin: low priority / non urgent issue, expect longer wait time for PR reviews O: tests Objective: Test more aspect of the relayer
Projects
None yet
Milestone
Refactor sink
Development

Successfully merging a pull request may close this issue.

Test ordered channel relaying with concurrent relayers informalsystems/hermes
2 participants
@soareschen @adizere