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 support for UpdateClient to hyperspace relayer abstraction #863

Closed
damiannolan opened this issue Nov 6, 2023 · 4 comments
Closed

Add support for UpdateClient to hyperspace relayer abstraction #863

damiannolan opened this issue Nov 6, 2023 · 4 comments

Comments

@damiannolan
Copy link
Contributor

damiannolan commented Nov 6, 2023

Summary / Feature Request

Add support for ibc client updates using the hyperspace relayer implementation.

Essentially, implement this:
https://github.com/strangelove-ventures/interchaintest/blob/main/relayer/hyperspace/hyperspace_commander.go#L224-L226

@damiannolan
Copy link
Contributor Author

I assume the hyperspace relayer must have client update functionality built in somehow as there is already successful happy path transfer tests using cosmos chains <-> polkadot. I'm guessing that it is just not directly exposed via a CLI?

@jonathanpberger
Copy link
Contributor

Doesn't look like there's an update client command exposed in the CLI. When that's added upstream, we'll revisit.

In the meantime, here's what's exposed: https://github.com/ComposableFi/composable-ibc/blob/master/hyperspace/README.md#running-hyperspace---cli-interface

@damiannolan
Copy link
Contributor Author

Thanks, I'll open an issue upstream as well 👍

@Reecepbcups
Copy link
Member

Once upstream adds it we are happy to add it in. Closing since it is out of our control for the time being.

@Reecepbcups Reecepbcups closed this as not planned Won't fix, can't repro, duplicate, stale May 10, 2024
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

No branches or pull requests

3 participants