Upgrade to Polkadot v0.9.42
Description
This is the upgrade from polkadot-v0.9.32 to polkadot-v0.9.42 that brings several breaking changes:
- Babe refactoring: https://github.com/paritytech/substrate/pull/13883
- Balances refactoring: https://github.com/paritytech/substrate/pull/12951
- Offences refactoring.
-
DustRemoval
type is incompatible withTreasury
.
Fixed errors
Will need refactoring
- The pallet
duniter-account
will need to be refactored for a better integration with the newAccountData
13ab4150. - Some scenarios (like 9d0c7835) are now deprecated due to the refactoring of the pallet balances that adds checks and providers limiting extrinsics usage.
- The providers, consumers, and sufficients counts need some refactoring to be better integrated with the pallet balances own new counts. 007aba57 introduces some workarounds to get the tests passing (with non blocking errors due to the providers count), but an overall spec reviewing and refactoring will be needed.
Changed (dependencies)
- Upgraded Duniter Substrate.
- Upgraded Duniter Subxt.
Changed
- Updated weight files.
- Updated tests with new Subxt API.
- Changed Babe sealing and RPC according to Babe refactoring.
- Updated pallet configurations according to Offences refactoring.
- Updated xtask.
Added
- Added
DustHandle
to convert the dust from pallet balances and add it to the treasury f357c9ec.
Deprecated
- One pallet balances benchmark needed https://github.com/duniter/substrate/commit/c36ab4f32454318a47777b24b6533c44121fc10b because pallet
duniter-account
was adding another provider, 007aba57 workaround deprecated this behavior.
To discuss
- Values setting for
HoldIdentifier
,FreezeIdentifier
,MaxHolds
,MaxFreezes
, and particularlyMaxSetIdSessionEntries
.
Edited by Benjamin Gallois
Merge request reports
Activity
assigned to @bgallois
added 1 commit
- c74de40f - fix ImplicitCallIndex and Weight::from_ref_time deprecation
- Resolved by Benjamin Gallois
The current state of this big upgrade is:
- Compilation without error or warning.
-
DustRemoval
andTreasury
are incompatible with 0.9.42, and this needs to be fixed. - Pallet
duniter-account
will need some refactoring because to pass palletbalances
benchmarks, the newflags: ExtraFlags
ofpallet_balance::AccountData
needs to be set either to new or old logic. AsExtraFlags
is private and only default to new logic, I implemented this very inelegant workaround for the moment. - Pallet
balances
benchmark failing with1 == 2
and is the only extrinsic to fail for all pallets. - Starting a node using
RUST_LOG=runtime=debug RUST_BACKTRACE=1 cargo run -- -lruntime=debug --dev
failed withERROR tokio-runtime-worker sc_service::task_manager: Essential task babe-worker failed. Shutting down service.
butRUST_LOG=runtime=debug RUST_BACKTRACE=1 cargo run -- -lruntime=debug --dev --sealing=instant
is working fine, so it seems related to Babe.
If @HugoTrentesaux or @tuxmain see something evident that I'm missing for 5 otherwise I will post on the Polkadot Stack Exchange.
Please register or sign in to reply