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

Post-MVP pre-launch IBC tasks #5199

Closed
1 of 7 tasks
cwgoes opened this issue Oct 15, 2019 · 10 comments
Closed
1 of 7 tasks

Post-MVP pre-launch IBC tasks #5199

cwgoes opened this issue Oct 15, 2019 · 10 comments
Assignees
Labels
S:blocked Status: Blocked

Comments

@cwgoes
Copy link
Contributor

cwgoes commented Oct 15, 2019

"MVP" is the first collection of code merged, and "launch" is the version proposed for Hub deployment.

@cwgoes cwgoes added this to the IBC Implementation & Integration milestone Oct 15, 2019
@tnachen
Copy link
Contributor

tnachen commented Oct 16, 2019

  • Blocked on evidence module -> That's ADR 009: Evidence Module #4826 right?
  • Update to canonical specification-defined encoding for provableStore data (blocked on spec) -> Do we have a github issue on the spec side?
  • Integrate with specification test vectors -> Do we have a github issue on the spec side?
  • ICS 20 bank-bridge module -> I'll create a Github issue then.

@cwgoes
Copy link
Contributor Author

cwgoes commented Oct 16, 2019

@cwgoes cwgoes mentioned this issue Oct 16, 2019
5 tasks
@cwgoes
Copy link
Contributor Author

cwgoes commented Oct 16, 2019

IRIS has started on the bank-bridge module, and I have reviewed it: #5204

@cwgoes
Copy link
Contributor Author

cwgoes commented Oct 16, 2019

@marbar3778 suggested that we call it the "bank packet handling module".

I tend to agree, "bridge module" evokes PoW bridges, which this is not.

@fedekunze fedekunze added the S:blocked Status: Blocked label Oct 21, 2019
@cwgoes
Copy link
Contributor Author

cwgoes commented Oct 22, 2019

I am working on the specification compliance test vectors in cosmos/ibc#291.

I expect an initial version sometime next week (when SFBW may cause busy-ness, so no rush) - at that point, it will be possible for the SDK to import a Go shim and test all the specified IBC functions against canonical test vectors.

@cwgoes
Copy link
Contributor Author

cwgoes commented Oct 28, 2019

Note that the SDK is currently implementing v1.0.0-rc4 of the IBC spec.

There is already a v1.0.0-rc5, and there will be another release candidate or two prior to finalisation.

I expect it will be most efficient for the SDK to test & polish the current version before upgrading to the latest spec; this is fine, they can proceed in parallel.

You can always find all the spec releases here.

@cwgoes
Copy link
Contributor Author

cwgoes commented Nov 5, 2019

Suggested additions by @zmanian et al.

  • Dynamic port allocation.
  • Solo client support

@jackzampolin
Copy link
Member

Can we consolidate this issue into #5125?

@jackzampolin
Copy link
Member

Closing this in favor of #5125 where I have also moved these items.

@cwgoes
Copy link
Contributor Author

cwgoes commented Nov 21, 2019

Thanks @jackzampolin, I updated #5125 a bit. I think it's comprehensive now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S:blocked Status: Blocked
Projects
None yet
Development

No branches or pull requests

6 participants