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

Set state to flushing complete after UpgradeAck completes (if possible) #4249

Closed
3 of 6 tasks
colin-axner opened this issue Aug 2, 2023 · 0 comments
Closed
3 of 6 tasks

Set state to flushing complete after UpgradeAck completes (if possible) #4249

colin-axner opened this issue Aug 2, 2023 · 0 comments
Assignees
@damiannolan @chatton
Labels
channel-upgradability Channel upgradability feature
Milestone
04-channel upgrad...

Comments

@colin-axner
Copy link
Contributor

colin-axner commented Aug 2, 2023 •
edited
Loading

Summary

Pre-requirements:

If UpgradeAck completes and there are no pending in flight packets, move the channel state to flushing complete. The Open handler will need to be updated to expect FLUSHCOMPLETE instead of UPGRADEACK

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@colin-axner colin-axner added the channel-upgradability Channel upgradability feature label Aug 2, 2023
@colin-axner colin-axner added this to the 04-channel upgrades alpha milestone Aug 2, 2023
@chatton chatton mentioned this issue Aug 17, 2023
Merged
9 tasks
@chatton chatton closed this as completed Aug 17, 2023
@github-project-automation github-project-automation bot moved this to Done in ibc-go Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@damiannolan damiannolan

@chatton chatton

Labels
channel-upgradability Channel upgradability feature
Projects
ibc-go
Archived in project
Milestone
04-channel upgrades alpha
Development

No branches or pull requests
3 participants
@damiannolan @chatton @colin-axner