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

Rename BagsList to VoterList and Add Score #5463

Merged
shawntabrizi merged 11 commits into from
May 19, 2022
Merged

Rename BagsList to VoterList and Add Score #5463

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
2f4827c
add migration for simple pallet rename
shawntabrizi May 5, 2022
aa4d52f
Merge branch 'master' into shawntabrizi-bags-to-voter-rename
shawntabrizi May 6, 2022
068a0bd
fmt
shawntabrizi May 6, 2022
040d442
also add score migration
shawntabrizi May 6, 2022
0d5d44f
Merge branch 'master' into shawntabrizi-bags-to-voter-rename
shawntabrizi May 9, 2022
dc80b8d
Merge branch 'master' into shawntabrizi-bags-to-voter-rename
shawntabrizi May 18, 2022
a9241d8
add try-runtime feature to bags list
shawntabrizi May 18, 2022
6e0b107
add pre_upgrade to pallet rename
shawntabrizi May 18, 2022
65c12b7
Merge remote-tracking branch 'origin/master' into shawntabrizi-bags-t…
May 19, 2022
da4d1f0
update lockfile for {"substrate"}
May 19, 2022
719f7bb
Merge branch 'master' into shawntabrizi-bags-to-voter-rename
shawntabrizi May 19, 2022
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
Prev Previous commit
Next Next commit
add pre_upgrade to pallet rename
  • Loading branch information
@shawntabrizi
shawntabrizi committed May 18, 2022
commit 6e0b10704b20a4f456f0ca246e9684ab6faf054b
8 changes: 7 additions & 1 deletion runtime/kusama/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1626,7 +1626,7 @@ pub type Executive = frame_executive::Executive<
AllPalletsWithSystem,
(
RenameBagsListToVoterList,
pallet_bags_list::migrations::AddScore<Runtime, ()>,
pallet_bags_list::migrations::AddScore<Runtime>,
InitiatePoolConfigs,
),
>;
Expand All @@ -1636,6 +1636,12 @@ pub type SignedPayload = generic::SignedPayload<Call, SignedExtra>;
/// A migration which renames the pallet `BagsList` to `VoterList`
pub struct RenameBagsListToVoterList;
impl frame_support::traits::OnRuntimeUpgrade for RenameBagsListToVoterList {
#[cfg(feature = "try-runtime")]
fn pre_upgrade() -> Result<(), &'static str> {
// For other pre-upgrade checks, we need the storage to already be migrated.
frame_support::storage::migration::move_pallet(b"BagsList", b"VoterList");
Comment on lines +1641 to +1642
Copy link
Member

@bkchr bkchr May 19, 2022

Choose a reason for hiding this comment

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

I don't get why we need this in the pre upgrade check?

Copy link
Contributor

@kianenigma kianenigma May 19, 2022

Choose a reason for hiding this comment

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

this is just a hack because we have two migrations around the same pallet. All pre-migrations are ran first, then all migrations, then all post migrations.

We need to mimic the old migrations happening to make it work.

Perhaps we can change the logic here to run the pre-post hooks per migration, not all at once.

Copy link
Member Author

@shawntabrizi shawntabrizi May 19, 2022

Choose a reason for hiding this comment

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

Yeah I was thinking the same thing, but not sure how easily implementable that is given the "tuple" trait stuff.

Ok(())
}
fn on_runtime_upgrade() -> frame_support::weights::Weight {
frame_support::storage::migration::move_pallet(b"BagsList", b"VoterList");
frame_support::weights::Weight::MAX
Expand Down
8 changes: 7 additions & 1 deletion runtime/polkadot/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1512,14 +1512,20 @@ pub type Executive = frame_executive::Executive<
frame_system::ChainContext<Runtime>,
Runtime,
AllPalletsWithSystem,
(RenameBagsListToVoterList, pallet_bags_list::migrations::AddScore<Runtime, ()>),
(RenameBagsListToVoterList, pallet_bags_list::migrations::AddScore<Runtime>),
>;
/// The payload being signed in transactions.
pub type SignedPayload = generic::SignedPayload<Call, SignedExtra>;

/// A migration which renames the pallet `BagsList` to `VoterList`
pub struct RenameBagsListToVoterList;
impl frame_support::traits::OnRuntimeUpgrade for RenameBagsListToVoterList {
#[cfg(feature = "try-runtime")]
fn pre_upgrade() -> Result<(), &'static str> {
// For other pre-upgrade checks, we need the storage to already be migrated.
frame_support::storage::migration::move_pallet(b"BagsList", b"VoterList");
Ok(())
}
fn on_runtime_upgrade() -> frame_support::weights::Weight {
frame_support::storage::migration::move_pallet(b"BagsList", b"VoterList");
frame_support::weights::Weight::MAX
Expand Down
8 changes: 7 additions & 1 deletion runtime/westend/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1161,14 +1161,20 @@ pub type Executive = frame_executive::Executive<
frame_system::ChainContext<Runtime>,
Runtime,
AllPalletsWithSystem,
(RenameBagsListToVoterList, pallet_bags_list::migrations::AddScore<Runtime, ()>),
(RenameBagsListToVoterList, pallet_bags_list::migrations::AddScore<Runtime>),
>;
/// The payload being signed in transactions.
pub type SignedPayload = generic::SignedPayload<Call, SignedExtra>;

/// A migration which renames the pallet `BagsList` to `VoterList`
pub struct RenameBagsListToVoterList;
impl frame_support::traits::OnRuntimeUpgrade for RenameBagsListToVoterList {
#[cfg(feature = "try-runtime")]
fn pre_upgrade() -> Result<(), &'static str> {
// For other pre-upgrade checks, we need the storage to already be migrated.
frame_support::storage::migration::move_pallet(b"BagsList", b"VoterList");
Ok(())
}
fn on_runtime_upgrade() -> frame_support::weights::Weight {
frame_support::storage::migration::move_pallet(b"BagsList", b"VoterList");
frame_support::weights::Weight::MAX
Expand Down