-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Conversation
/benchmark xcm kusama pallet_xcm_benchmarks::generic |
Benchmark Kusama XCM for branch "shawntabrizi-kusama-xcm-benchmarks" with command cargo run --quiet --release --features=runtime-benchmarks -- benchmark --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::generic --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_generic.rs Results
ERROR: Unable to commit file ./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_generic.rs |
/benchmark xcm kusama pallet_xcm_benchmarks::generic |
Benchmark Kusama XCM for branch "shawntabrizi-kusama-xcm-benchmarks" with command cargo run --quiet --release --features=runtime-benchmarks -- benchmark --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::generic --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_generic.rs Results
|
…k --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::generic --extrinsic=* --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_generic.rs
/benchmark xcm kusama pallet_xcm_benchmarks::fungible |
Benchmark Kusama XCM for branch "shawntabrizi-kusama-xcm-benchmarks" with command cargo run --quiet --release --features=runtime-benchmarks -- benchmark --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::fungible --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_fungible.rs Results
|
…k --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::fungible --extrinsic=* --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_fungible.rs
/benchmark xcm kusama pallet_xcm_benchmarks::generic |
/benchmark xcm kusama pallet_xcm_benchmarks::fungible |
Benchmark Kusama XCM for branch "shawntabrizi-kusama-xcm-benchmarks" with command cargo run --quiet --profile=production --features=runtime-benchmarks -- benchmark --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::fungible --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_fungible.rs Toolchain: stable-x86_64-unknown-linux-gnu (default) Results
ERROR: Unable to commit file ./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_fungible.rs |
/benchmark xcm kusama pallet_xcm_benchmarks::fungible |
Benchmark Kusama XCM for branch "shawntabrizi-kusama-xcm-benchmarks" with command cargo run --quiet --profile=production --features=runtime-benchmarks -- benchmark --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::fungible --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_fungible.rs Toolchain: stable-x86_64-unknown-linux-gnu (default) Results
|
…shawntabrizi-kusama-xcm-benchmarks
…-- benchmark --chain=kusama-dev --steps=50 --repeat=20 --pallet=pallet_xcm_benchmarks::fungible --extrinsic=* --execution=wasm --wasm-execution=compiled --heap-pages=4096 --header=./file_header.txt --template=./xcm/pallet-xcm-benchmarks/template.hbs --output=./runtime/kusama/src/weights/xcm/pallet_xcm_benchmarks_fungible.rs
// --execution=wasm | ||
// --wasm-execution=compiled | ||
// --heap-pages=4096 | ||
// --header=./file_header.txt |
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.
Where does benchbot get this file? I thought it was from the top-level file_header.txt
file, but I've already modified it to have the current year in the license, yet this autogenerated file STILL says 2021. What gives?
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.
it is def the top level file header
I also approve this PR |
// Storage: XcmPallet SupportedVersion (r:1 w:0) | ||
// Storage: XcmPallet VersionDiscoveryQueue (r:1 w:1) | ||
// Storage: XcmPallet SafeXcmVersion (r:1 w:0) | ||
// Storage: Configuration ActiveConfig (r:1 w:0) |
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.
I think this storage item should be whitelisted since it is read in all blocks afaik.
Oh wow, since this touches a runtime directory, we'll need an additional approval, either from @bkchr or @gavofyork (or @rphmeier if he's ok with reviewing this). |
bot merge |
Error: Checks failed for 00d77ff |
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.
I'm only reviewing the changes in kusama/lib.rs and kusama/xcm_config.rs to ensure they're not causing any issues.
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.
I also only have reviewed the runtime lib.rs
Let's not merge this until we have deployed it to Westend for a bit.