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

Add packet forwarding middleware tests #1983

Closed
adizere opened this issue Mar 21, 2022 · 0 comments · Fixed by #2849
Closed

Add packet forwarding middleware tests #1983

adizere opened this issue Mar 21, 2022 · 0 comments · Fixed by #2849
Assignees
Labels
I: infrastructure Internal: related to Infrastructure (testing, deployment, etc) I: protocol Internal: related to IBC protocol changes (eg. v2 update) O: tests Objective: Test more aspect of the relayer
Milestone

Comments

@adizere
Copy link
Member

adizere commented Mar 21, 2022

Note: This is a suggestions that came up in a conversation with @soareschen .

Context

The IBC packet forward middleware v1.0.1 was added to the Hub features in the Vega upgrade
https://github.com/cosmos/gaia/blob/main/docs/roadmap/cosmos-hub-roadmap-2.0.md#vega-upgrade-completed-december-14-2021

Problem statement

There are a couple of problems with this:

  • There are no integration tests for this feature.
  • It is unclear to the members of our team how the feature works.

Furthermore, it seems that the packet forwarding middleware is a potential solution to address some of the scalability problems in Cosmos, suggesting we should develop the expertise around it and understand it more deeply.

Acceptance criteria

  • Write an integration test and document the functionality.
@adizere adizere added this to the v0.15.0 milestone Mar 21, 2022
@adizere adizere added I: dependencies Internal: related to dependencies O: tests Objective: Test more aspect of the relayer I: infrastructure Internal: related to Infrastructure (testing, deployment, etc) I: protocol Internal: related to IBC protocol changes (eg. v2 update) labels Mar 21, 2022
@adizere adizere modified the milestones: v0.15.0, v1.1 Apr 26, 2022
@adizere adizere modified the milestones: v1.1, v1.2 Jun 29, 2022
@romac romac removed the I: dependencies Internal: related to dependencies label Aug 3, 2022
@adizere adizere added the P-high label Oct 31, 2022
Repository owner moved this from Backlog to Closed in IBC-rs: the road to v1 Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: infrastructure Internal: related to Infrastructure (testing, deployment, etc) I: protocol Internal: related to IBC protocol changes (eg. v2 update) O: tests Objective: Test more aspect of the relayer
Projects
No open projects
Status: Closed
Development

Successfully merging a pull request may close this issue.

3 participants