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

Remove persisted portID's for interchain accounts #4638

Open
3 tasks
colin-axner opened this issue Sep 12, 2023 · 0 comments
Open
3 tasks

Remove persisted portID's for interchain accounts #4638

colin-axner opened this issue Sep 12, 2023 · 0 comments
Labels
27-interchain-accounts needs discussion Issues that need discussion before they can be worked on
Milestone

Comments

@colin-axner
Copy link
Contributor

Summary

Remove persisted state from interchain accounts

Problem Definition

Upon init genesis, ics27 persists information such as the portID that is never referenced by during regular processes. It might have been thought that this information is required to regenerate capabilities upon a genesis restart, but the capability module will handle this automatically

Proposal

Remove unnecessary persisted portIDs in host/controller. We may consider writing a small migration handler to remove existing state. Otherwise the setPort functions can be removed from the host and controller submodules.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@colin-axner colin-axner added needs discussion Issues that need discussion before they can be worked on 27-interchain-accounts labels Sep 12, 2023
@colin-axner colin-axner moved this to Backlog 🕐 in ibc-go Sep 9, 2024
@colin-axner colin-axner added this to the v11.0.0 milestone Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
27-interchain-accounts needs discussion Issues that need discussion before they can be worked on
Projects
Status: Backlog 🕐
Development

No branches or pull requests

1 participant