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

wip: penumbra relayer provider #2

Closed
wants to merge 8 commits into from
Closed

wip: penumbra relayer provider #2

wants to merge 8 commits into from

Conversation

avahowell
Copy link

this PR adds the basic, work in progress, relayer provider and chain processor for penumbra.

Everything up to packet relaying is currently tested and works. This is a fork of the cosmos provider & processor, so there may be some dead code and code duplication.

@avahowell
Copy link
Author

avahowell commented Oct 26, 2022

Pulled this to latest main and resolved conflicts. Remaining tasks include:

  • Pull over changes for supporting non-default prefix
  • Update IBC protos to v5 (in penumbra) (NOTE: was this done when we imported ibc-rs upstream? cc @erwanor )
  • Resolve remaining build errors

@erwanor
Copy link
Member

erwanor commented Dec 8, 2022

We are using the v5 ibc proto bindings through ibc_proto (the crate) but I just checked the one in proto/ibc-go-vendor and they look like the v3s. I'm not sure what we use the latter ones for, or how much work would that be to update them. Happy to pair on this if you want, or even take this on @avahowell

@conorsch
Copy link

Superseded by cosmos#1144 ; see penumbra-zone/penumbra#465 for details.

@conorsch conorsch closed this Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants