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

add nomination pools to westend runtime #5405

Merged
merged 4 commits into from
Apr 28, 2022
Merged

Conversation

kianenigma
Copy link
Contributor

Will have a sudo call that follows and sets some of the configs.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Apr 27, 2022
@kianenigma kianenigma added B7-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Apr 27, 2022
@shawntabrizi shawntabrizi changed the title add nomination pools to westend rutnime add nomination pools to westend runtime Apr 27, 2022
@kianenigma
Copy link
Contributor Author

/benchmark runtime westend pallet pallet_nomination_pools

@parity-benchapp
Copy link

parity-benchapp bot commented Apr 28, 2022

Benchmark Runtime Westend Pallet for branch "kiz-pools-wnd" with command cargo run --quiet --profile=production --features=runtime-benchmarks -- benchmark pallet --chain=westend-dev --steps=50 --repeat=20 --pallet=pallet pallet_nomination_pools --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --output=./runtime/westend/src/weights/

Toolchain: stable-x86_64-unknown-linux-gnu (default)
rustc 1.57.0 (f1edd0429 2021-11-29)

Results
error[E0700]: hidden type for `impl Trait` captures lifetime that does not appear in bounds
   --> node/core/backing/src/lib.rs:190:6
    |
190 | ) -> Result<(), Error>
    |      ^^^^^^^^^^^^^^^^^
    |
note: hidden type `impl futures::Future` captures lifetime smaller than the function body
   --> node/core/backing/src/lib.rs:190:6
    |
190 | ) -> Result<(), Error>
    |      ^^^^^^^^^^^^^^^^^

For more information about this error, try `rustc --explain E0700`.
error: could not compile `polkadot-node-core-backing` due to previous error
error: build failed

ERROR: Unable to commit file ./runtime/westend/src/weights/

@kianenigma
Copy link
Contributor Author

/benchmark runtime westend pallet pallet_nomination_pools

1 similar comment
@kianenigma
Copy link
Contributor Author

/benchmark runtime westend pallet pallet_nomination_pools

@kianenigma
Copy link
Contributor Author

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Statuses failed for c7c9eda

@kianenigma
Copy link
Contributor Author

bot merge

@paritytech-processbot paritytech-processbot bot merged commit e0fc2b7 into master Apr 28, 2022
@paritytech-processbot paritytech-processbot bot deleted the kiz-pools-wnd branch April 28, 2022 18:16
coderobe pushed a commit that referenced this pull request Apr 29, 2022
* add nomination pools to westend rutnime

* Fix buikd

* fit some shit

* add real weights from substrate weights computed on the same bench machine
bkchr pushed a commit that referenced this pull request Apr 29, 2022
* add nomination pools to westend rutnime

* Fix buikd

* fit some shit

* add real weights from substrate weights computed on the same bench machine
@coderobe
Copy link
Contributor

coderobe commented May 3, 2022

The westend runtime containing this has been deployed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants