Skip to content

feat: Update substrate to v0.9.43#3446

Closed
ukint-vs wants to merge 67 commits intomasterfrom
vs/substrate-v0.9.43-2
Closed

feat: Update substrate to v0.9.43#3446
ukint-vs wants to merge 67 commits intomasterfrom
vs/substrate-v0.9.43-2

Conversation

@ukint-vs
Copy link
Member

@ukint-vs ukint-vs commented Oct 24, 2023

update substrate to v0.9.42:

update substrate to v0.9.43:

  • add weights in pallet_sudo
  • update pallet-staking-rewards tests (stash is controller from now on)
  • Add migration to set current pallet versions to pallets that were added after genesis

todo:

  • update weights
  • fix balances weights

@gear-tech/dev

@ukint-vs ukint-vs force-pushed the vs/substrate-v0.9.43-2 branch from e1a52ca to 3510f31 Compare October 29, 2023 15:37
@ukint-vs ukint-vs requested review from breathx and ekovalev October 29, 2023 15:44
RuntimeApiCollection<StateBackend = sc_client_api::StateBackendFor<FullBackend, Block>>,
ExecutorDispatch: NativeExecutionDispatch + 'static,
{
if config.keystore_remote.is_some() {
Copy link
Member

Choose a reason for hiding this comment

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

Does remote keystore now supported? hm

pub struct NominationPoolsMigrationV4OldPallet;
impl Get<Perbill> for NominationPoolsMigrationV4OldPallet {
fn get() -> Perbill {
Perbill::from_percent(10)
Copy link
Member

Choose a reason for hiding this comment

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

any comments on that?

Copy link
Member Author

Choose a reason for hiding this comment

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

}
}

// DustRemoval handler
Copy link
Member

Choose a reason for hiding this comment

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

what was the problem of previous impl? trait bounds? than add todo to remove initial one

Copy link
Member Author

@ukint-vs ukint-vs Oct 30, 2023

Choose a reason for hiding this comment

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

Conflicting implementation. Since balances::DustHandler switched to fungible::Credit from Imbalance. But we are using Imbalance in fn align_supply

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, add todo pls

Copy link
Member Author

Choose a reason for hiding this comment

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

We don't need Todo. Both structs are for two different implementation and purposes

Copy link
Member

@gshep gshep left a comment

Choose a reason for hiding this comment

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

🥇

@ukint-vs ukint-vs force-pushed the vs/substrate-v0.9.43-2 branch from 5e65f91 to 5a182a7 Compare November 7, 2023 18:59
@ukint-vs ukint-vs force-pushed the vs/substrate-v0.9.43-2 branch from 5a182a7 to 2f3ecab Compare November 7, 2023 19:02
Copy link
Contributor

@shamilsan shamilsan left a comment

Choose a reason for hiding this comment

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

Big deal 👍

@NikVolf
Copy link
Member

NikVolf commented Nov 8, 2023

This should go in separate release

Copy link
Member

@NikVolf NikVolf left a comment

Choose a reason for hiding this comment

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

Isolated testable release for it please

@breathx breathx added A5-dontmerge PR should not be merged yet and removed A0-pleasereview PR is ready to be reviewed by the team labels Nov 10, 2023
@NikVolf
Copy link
Member

NikVolf commented Dec 1, 2023

Can we finally merge it and make update to 0.43 ASAP?

@ukint-vs ukint-vs added the A0-pleasereview PR is ready to be reviewed by the team label Dec 1, 2023
@breathx
Copy link
Member

breathx commented Dec 11, 2023

To be merged into separate release branch, isn't it? @ukint-vs

@breathx breathx closed this Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A0-pleasereview PR is ready to be reviewed by the team A5-dontmerge PR should not be merged yet D2-node Gear Node

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants