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

Rococo AH: cleanup storage #4444

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Rococo AH: cleanup storage #4444

wants to merge 2 commits into from

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented May 13, 2024

Follow up on #4414 to clean up the old storage.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez added the T14-system_parachains This PR/Issue is related to system parachains. label May 13, 2024
Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ignored this on purpose, because these chains are going to stop in the hopefully near future any way :D

@kianenigma
Copy link
Contributor

How about a

pub struct DetectDanglingStorage<Exceptions: Get<Vec<u8>>>;
impl OnRuntimeUpgrade for DetectDanglingStorage {
  fn on_runtime_upgrade() { noop } 
  fn post_update() { 
    // iterate over all keys, match them to pallet storage from metadata
    // using the same tools that make `DecodeEntireState`
    // detect all keys that have value and
    // 1. not part of any pallet
    // 2. not part of `Exceptions`
  }

@ggwpez
Copy link
Member Author

ggwpez commented May 13, 2024

Yea ^ that sounds pretty nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T14-system_parachains This PR/Issue is related to system parachains.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants