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

Timing out packets on ordered channels when in flushing state #4454

Closed
damiannolan opened this issue Aug 24, 2023 · 1 comment
Closed

Timing out packets on ordered channels when in flushing state #4454

damiannolan opened this issue Aug 24, 2023 · 1 comment
Assignees
Labels
04-channel channel-upgradability Channel upgradability feature

Comments

@damiannolan
Copy link
Member

Originally came up in #4425.

Timing out packets on an ordered channel normally results in a channel state of CLOSED.

Aborting an upgrade if the upgrade timeout has passed results in an OPEN channel with an error receipt written AND upgrade state is removed.

What is the desired state of a channel end after a packet on an ordered channel times out when in flushing?

We could allow timing out packets when flushing on ordered channels, however if application logic is dependent on strict ordering then we may end up in bad/undesired states.

Adding needs discussion label

Originally posted by @damiannolan in #4425 (comment)

@damiannolan damiannolan added 04-channel channel-upgradability Channel upgradability feature labels Aug 24, 2023
@damiannolan damiannolan added this to the 04-channel upgrades beta milestone Aug 24, 2023
@damiannolan damiannolan added the needs discussion Issues that need discussion before they can be worked on label Aug 24, 2023
@damiannolan
Copy link
Member Author

After discussion we've determined the desired end state here is likely:

  • Packet is timed out successfully
  • Channel upgrade is aborted and upgrade info deleted
  • Channel is now in CLOSED state

cc. @AdityaSripal for visibility

@damiannolan damiannolan removed the needs discussion Issues that need discussion before they can be worked on label Aug 28, 2023
@damiannolan damiannolan moved this to Todo in ibc-go Aug 28, 2023
@chatton chatton self-assigned this Aug 28, 2023
@chatton chatton closed this as completed Aug 30, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done in ibc-go Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
04-channel channel-upgradability Channel upgradability feature
Projects
Archived in project
Development

No branches or pull requests

2 participants