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

ibc client upgrades #3073

Merged
merged 2 commits into from
Oct 6, 2023
Merged

ibc client upgrades #3073

merged 2 commits into from
Oct 6, 2023

Conversation

avahowell
Copy link
Contributor

@avahowell avahowell commented Sep 21, 2023 •
edited by erwanor
Loading

Closes #635

@avahowell avahowell temporarily deployed to smoke-test September 21, 2023 21:35 — with GitHub Actions Inactive
@avahowell avahowell changed the title add ClientUpgradeProofVerifier [WIP] ibc client upgrades Sep 21, 2023
@avahowell avahowell marked this pull request as draft September 21, 2023 21:35
@conorsch
Copy link
Contributor

@avahowell What's the status of this PR? Mergeable for 61 (#3046), or still more work required?

@hdevalence
Copy link
Member

I think we should delay it until the next cycle.

@avahowell
Copy link
Contributor Author

avahowell commented Sep 25, 2023 •
edited
Loading

Yes, this PR only contains the proof verification as of now. we still need to add the message itself and its state machine handling

@avahowell avahowell force-pushed the ibc-client-upgrades branch from 7b8bd51 to 55bedc7 Compare October 6, 2023 19:56
@avahowell avahowell force-pushed the ibc-client-upgrades branch from 55bedc7 to de753c9 Compare October 6, 2023 19:57
@avahowell avahowell temporarily deployed to smoke-test October 6, 2023 19:57 — with GitHub Actions Inactive
@avahowell avahowell marked this pull request as ready for review October 6, 2023 19:57
@avahowell avahowell changed the title [WIP] ibc client upgrades ibc client upgrades Oct 6, 2023
@conorsch
Copy link
Contributor

conorsch commented Oct 6, 2023

Merging! This'll ship in 62.

@conorsch conorsch merged commit ace2278 into main Oct 6, 2023
8 checks passed
@conorsch conorsch deleted the ibc-client-upgrades branch October 6, 2023 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ibc: implement upgradeclient to allow counterparty chains to upgrade
3 participants
@avahowell @conorsch @hdevalence