-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Add nomination pools to Polkadot runtime #5582
Conversation
/bench runtime polkadot pallet_nomination_pools |
Benchmark Runtime Polkadot Pallet for branch "kiz-pools-polkadot" with command cargo run --quiet --profile=production --features=runtime-benchmarks -- benchmark pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --output=./runtime/polkadot/src/weights/ Toolchain: stable-x86_64-unknown-linux-gnu (default) Results
ERROR: Unable to commit file ./runtime/polkadot/src/weights/ |
/bench runtime polkadot pallet_nomination_pools |
Benchmark Runtime Polkadot Pallet for branch "kiz-pools-polkadot" with command cargo run --quiet --profile=production --features=runtime-benchmarks -- benchmark pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --output=./runtime/polkadot/src/weights/ Toolchain: stable-x86_64-unknown-linux-gnu (default) Results
ERROR: Unable to commit file ./runtime/polkadot/src/weights/ |
I think you currently have to copy&paste the benchmark command without the |
/benchmark runtime custom pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --output=./runtime/polkadot/src/weights/ |
Benchmark Runtime Custom for branch "kiz-pools-polkadot" with command cargo run --quiet --profile=production --features runtime-benchmarks -- benchmark pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --output=./runtime/polkadot/src/weights/ Toolchain: stable-x86_64-unknown-linux-gnu (default) Results
ERROR: Unable to commit file ./runtime/polkadot/src/weights/ |
/benchmark runtime custom pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --header=./file_header.txt --output=./runtime/polkadot/src/weights/ |
Benchmark Runtime Custom for branch "kiz-pools-polkadot" with command cargo run --quiet --profile=production --features runtime-benchmarks -- benchmark pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --header=./file_header.txt --output=./runtime/polkadot/src/weights/ Toolchain: stable-x86_64-unknown-linux-gnu (default) Results
|
…kiz-pools-polkadot
…-- benchmark pallet --chain=polkadot-dev --steps=50 --repeat=20 --pallet=pallet_nomination_pools --extrinsic=* --execution=wasm --wasm-execution=compiled --header=./file_header.txt --output=./runtime/polkadot/src/weights/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When does this plan to go out?
I feel we should probably let this bake on Kusama a bit longer, especially given the recent bug reports.
Sounds like a good plan to me. |
@kianenigma Command |
bot rebase |
Branch is already up-to-date |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exciting! Lets go 🚀
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
…nto kiz-pools-polkadot
bot merge |
Waiting for commit status. |
Merge cancelled due to error. Error: Statuses failed for 36e7766 |
bot merge |
* master: Transaction payment runtime apis: query call info and fee details (#5871) [ci] Improve cancel-pipeline job (#5874) Bump wasmtime from 0.38.1 to 0.38.3 (#5802) Incorporate changes from substrate PR #11908 (#5815) Add nomination pools to Polkadot runtime (#5582) Change request-response protocol names to include genesis hash & fork id (#5870) [ci] Run check-runtime only for PRs (#5858) Bump tokio from 1.18.2 to 1.19.2 (#5678) Zombienet: test disputes with malus garbage candidates (#5857) add unit tests to run runtime migrations (#5865) Trivial networking changes for Substrate PR #11940 (#5841) Renaming CLI prunning and keep-blocks flags (#5863) Update yamux to fix a potential crash (#5861)
#### Intro Upgrade polkadot from v0.9.27 to v0.9.30 as a checkpoint, then to v0.9.33 (latest version without workspace dependencies) Require ComposableFi/composable-ibc#176 to be merged first. #### Migrations - v0.9.28 - [x] paritytech/polkadot#5582 - Nomination not present in our runtimes - v0.9.29 - [x] paritytech/substrate#12095 - Nomination not present in our runtimes - v0.9.30 - [x] paritytech/substrate#12034 - BagList/Staking not present in our runtimes - [x] paritytech/polkadot#5930 - Nomination/BagList/Staking not present in our runtimes - [x] paritytech/substrate#12230 - Staking not present in our runtimes - [x] paritytech/polkadot#5996 - Staking not present in our runtimes - [x] paritytech/substrate#12083 - Contracts not present in our runtimes Signed-off-by: cor <cor@pruijs.dev> Co-authored-by: cor <cor@pruijs.dev>
targeting release 0.9.23
Also sets the initial configs of the pools:
5 DOT to join a pool.
100 DOT to create a pool.
64 initial pools: only for initial safety: can be set to infinity when needed.
4096 members per pool: only for initial safety: can be set to infinity when needed.
128k total pool members: only for initial safety: can be set to infinity when needed.
Should only be merged once the following is merged on substrate:
UPDATE:
The above numbers are still my suggestion, but I this PR as it is has all the max counters set to 0, meaning that we can deploy this but no pool can be created. This will be helpful in making the process of enabling pools independent of a runtime upgrade.