Skip to content

Conversation

@nkpar
Copy link
Contributor

@nkpar nkpar commented Dec 18, 2025

Summary

The state trie migration from V0 to V1 format has been completed on Westend Asset Hub:

Chain Top Items Migrated Child Items Status
Westend Asset Hub 2,082,626 28,095 ✅ Complete

This PR removes the pallet-state-trie-migration from the runtime and adds a RemovePallet migration to clean up the storage.

Changes

  • Removed pallet-state-trie-migration dependency from Cargo.toml
  • Removed pallet configuration (impl pallet_state_trie_migration::Config)
  • Removed from construct_runtime! macro (index 70)
  • Added RemovePallet<StateTrieMigrationName, DbWeight> migration
  • Removed MigController and RootMigController accounts and tests

Related PRs

Test Plan

  • CI passes
  • RemovePallet migration tested via try-runtime (if enabled)

The state trie migration from V0 to V1 format has been completed on
Westend Asset Hub (2,082,626 top items + 28,095 child items).

This PR removes the pallet-state-trie-migration from the runtime and adds
a RemovePallet migration to clean up the storage.

Changes:
- Removed pallet-state-trie-migration dependency from Cargo.toml
- Removed pallet configuration (impl pallet_state_trie_migration::Config)
- Removed from construct_runtime! macro (index 70)
- Added RemovePallet<StateTrieMigrationName, DbWeight> migration
- Removed MigController and RootMigController accounts and tests
@nkpar nkpar requested review from bkchr and kianenigma December 18, 2025 16:27
@github-actions
Copy link
Contributor

Review required! Latest push from author must always be reviewed

@nkpar nkpar self-assigned this Dec 18, 2025
@nkpar nkpar added the T2-pallets This PR/Issue is related to a particular pallet. label Dec 18, 2025
@nkpar nkpar requested a review from a team as a code owner December 18, 2025 22:04
Replace deprecated polkadot.com URLs (returning 404 after site redesign)
with current equivalents:

- bags-list: staking-update-september-2021 -> wiki.polkadot.com/docs/learn-staking-advanced#bags-list
- fast-unstake: staking-update-february-2022 -> wiki.polkadot.com/learn/learn-nominator
- favicon: Use existing paritytech PNG logo (ICO was removed)
- papers: polkadot.com/papers -> github.com/polkadot-io/polkadot-white-paper
- approval checking blog: polkadot.com/blog -> wiki.polkadot.com/docs/learn-parachains-protocol
@nkpar nkpar force-pushed the remove-state-trie-migration-westend branch from 6985d17 to b3ec145 Compare December 18, 2025 22:21
@paritytech paritytech deleted a comment from cla-bot-2021 bot Dec 19, 2025
pull bot pushed a commit to cshein45/polkadot-sdk that referenced this pull request Jan 8, 2026
Related: paritytech#10694, paritytech#10515

Migrates all `exchange_asset` tests from the integration test suite to
unit tests under the `AssetHubWestend` runtime module using
`asset-test-utils`. Creates a reusable
`exchange_asset_on_asset_hub_works` helper function reducing duplication
and improving maintainability.

---------

Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Raymond Cheung <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T2-pallets This PR/Issue is related to a particular pallet.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants