Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Remove state migration from westend runtime. #6737

Merged
paritytech-processbot merged 10 commits into from
Feb 22, 2023
Merged

Remove state migration from westend runtime. #6737

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
b1d55c6
clean state_trie_migration from westend
cheme Feb 17, 2023
893f458
update cargo.lock
cheme Feb 17, 2023
10819c8
Revert "clean state_trie_migration from westend"
cheme Feb 17, 2023
0d0ef73
cleanup.
cheme Feb 17, 2023
5289745
use storage_alias
cheme Feb 17, 2023
d4c688a
Revert "use storage_alias"
cheme Feb 17, 2023
6b53ed0
Revert "Revert "use storage_alias""
cheme Feb 17, 2023
71b2f32
Force alias.
cheme Feb 17, 2023
e4321f0
magic
cheme Feb 17, 2023
e46e6f8
Merge branch 'master' into clean_westend_mig
cheme Feb 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion runtime/westend/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,6 @@ xcm-builder = { package = "xcm-builder", path = "../../xcm/xcm-builder", default
hex-literal = "0.3.4"
tiny-keccak = { version = "2.0.2", features = ["keccak"] }
keyring = { package = "sp-keyring", git = "https://github.com/paritytech/substrate", branch = "master" }
sp-trie = { git = "https://github.com/paritytech/substrate", branch = "master" }
serde_json = "1.0.81"
remote-externalities = { git = "https://github.com/paritytech/substrate", branch = "master", package = "frame-remote-externalities" }
tokio = { version = "1.24.2", features = ["macros"] }
Expand Down
34 changes: 13 additions & 21 deletions runtime/westend/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1243,7 +1243,7 @@ impl Get<&'static str> for StakingMigrationV11OldPallet {
///
/// Should be cleared after every release.
pub type Migrations = (
init_state_migration::InitMigrate,
clean_state_migration::CleanMigrate,
// "Use 2D weights in XCM v3" <https://github.com/paritytech/polkadot/pull/6134>
pallet_xcm::migration::v1::MigrateToV1<Runtime>,
parachains_ump::migration::v1::MigrateToV1<Runtime>,
Expand Down Expand Up @@ -1908,43 +1908,35 @@ mod remote_tests {
}
}

mod init_state_migration {
mod clean_state_migration {
use super::Runtime;
use frame_support::traits::OnRuntimeUpgrade;
use pallet_state_trie_migration::{AutoLimits, MigrationLimits, MigrationProcess};
use pallet_state_trie_migration::{AutoLimits, MigrationProcess, SignedMigrationMaxLimits};
#[cfg(not(feature = "std"))]
use sp_std::prelude::*;

/// Initialize an automatic migration process.
pub struct InitMigrate;
impl OnRuntimeUpgrade for InitMigrate {
pub struct CleanMigrate;
impl OnRuntimeUpgrade for CleanMigrate {
#[cfg(feature = "try-runtime")]
fn pre_upgrade() -> Result<Vec<u8>, &'static str> {
frame_support::ensure!(
AutoLimits::<Runtime>::get().is_none(),
"Automigration already started."
);
Ok(Default::default())
}

fn on_runtime_upgrade() -> frame_support::weights::Weight {
if MigrationProcess::<Runtime>::get() == Default::default() &&
AutoLimits::<Runtime>::get().is_none()
{
AutoLimits::<Runtime>::put(Some(MigrationLimits { item: 160, size: 204800 }));
log::info!("Automatic trie migration started.");
<Runtime as frame_system::Config>::DbWeight::get().reads_writes(2, 1)
} else {
log::info!("Automatic trie migration not started.");
<Runtime as frame_system::Config>::DbWeight::get().reads(2)
}
MigrationProcess::<Runtime>::kill();
AutoLimits::<Runtime>::kill();
SignedMigrationMaxLimits::<Runtime>::kill();
Copy link
Member

@bkchr bkchr Feb 17, 2023

Choose a reason for hiding this comment

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

You can use storage_alias to have the definition of them without including the pallet.

<Runtime as frame_system::Config>::DbWeight::get().writes(3)
}

#[cfg(feature = "try-runtime")]
fn post_upgrade(_state: Vec<u8>) -> Result<(), &'static str> {
frame_support::ensure!(
AutoLimits::<Runtime>::get().is_some(),
"Automigration started."
!MigrationProcess::<Runtime>::exists() &&
!AutoLimits::<Runtime>::exists() &&
!SignedMigrationMaxLimits::<Runtime>::exists(),
"State migration clean.",
);
Ok(())
}
Expand Down