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

Adds ability to prepare/initialize before running set_code benchmark#14435

Merged
paritytech-processbot[bot] merged 14 commits intomasterfrom
bko-set_code-benchmark-helper
Jun 23, 2023
Merged

Adds ability to prepare/initialize before running set_code benchmark#14435
paritytech-processbot[bot] merged 14 commits intomasterfrom
bko-set_code-benchmark-helper

Conversation

@bkontur
Copy link
Contributor

@bkontur bkontur commented Jun 21, 2023

See detailed description here: paritytech/cumulus#2765

Closes: paritytech/cumulus#2765

Cumulus companion: paritytech/cumulus#2766

@bkontur bkontur added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jun 21, 2023
@bkontur
Copy link
Contributor Author

bkontur commented Jun 21, 2023

bot bench $ pallet dev frame-system
bot bench $ pallet dev frame-system-benchmarking

@command-bot
Copy link

command-bot bot commented Jun 21, 2023

@bkontur https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3046481 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system. Check out https://gitlab.parity.io/parity/mirrors/substrate/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 1-6f403e28-2ed1-49c5-a5de-c410e705e17d to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Jun 21, 2023

@bkontur https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3046482 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system-benchmarking. Check out https://gitlab.parity.io/parity/mirrors/substrate/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 2-4dabaa89-dca6-43ca-8d4a-04b9dcec557e to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Jun 21, 2023

@bkontur Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system has finished. Result: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3046481 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3046481/artifacts/download.

@command-bot
Copy link

command-bot bot commented Jun 21, 2023

@bkontur Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system-benchmarking has finished. Result: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3046482 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3046482/artifacts/download.

@bkontur bkontur added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit label Jun 22, 2023
@bkontur
Copy link
Contributor Author

bkontur commented Jun 22, 2023

bot bench $ pallet dev frame-system

@command-bot
Copy link

command-bot bot commented Jun 22, 2023

@bkontur https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3049124 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system. Check out https://gitlab.parity.io/parity/mirrors/substrate/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 1-13610fb0-3041-47a3-a8d8-a0603647936f to cancel this command or bot cancel to cancel all commands in this pull request.

@bkontur bkontur marked this pull request as ready for review June 22, 2023 09:07
@bkontur bkontur requested review from a team June 22, 2023 09:07
@command-bot
Copy link

command-bot bot commented Jun 22, 2023

@bkontur Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system has finished. Result: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3049124 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3049124/artifacts/download.

Comment on lines +37 to +41
/// Adds ability to the Runtime to test against their sample code.
/// Default is `../res/kitchensink_runtime.compact.compressed.wasm`.
fn set_code_data() -> Vec<u8> {
include_bytes!("../res/kitchensink_runtime.compact.compressed.wasm").to_vec()
}
Copy link
Member

Choose a reason for hiding this comment

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

This is a very very good idea! :)

@bkontur
Copy link
Contributor Author

bkontur commented Jun 22, 2023

bot bench $ pallet dev frame-system

@command-bot
Copy link

command-bot bot commented Jun 22, 2023

@bkontur https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3050145 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system. Check out https://gitlab.parity.io/parity/mirrors/substrate/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 2-d64ed87d-7cc8-4e72-86b9-7cfbcf1d7cad to cancel this command or bot cancel to cancel all commands in this pull request.

@command-bot
Copy link

command-bot bot commented Jun 22, 2023

@bkontur Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet dev frame-system has finished. Result: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3050145 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3050145/artifacts/download.

@ggwpez
Copy link
Member

ggwpez commented Jun 22, 2023

bot bench-vm $ pallet dev frame-system

@command-bot
Copy link

command-bot bot commented Jun 22, 2023

@ggwpez https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3051678 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench-vm/bench-vm.sh" pallet dev frame-system. Check out https://gitlab.parity.io/parity/mirrors/substrate/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 3-e9831320-64b0-4f0e-a0c9-de4922bb4637 to cancel this command or bot cancel to cancel all commands in this pull request.

Copy link
Member

@ggwpez ggwpez left a comment

Choose a reason for hiding this comment

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

Need to wait for the new bench cmd to finish, then its good to go 🚀

@command-bot
Copy link

command-bot bot commented Jun 22, 2023

@ggwpez Command "$PIPELINE_SCRIPTS_DIR/commands/bench-vm/bench-vm.sh" pallet dev frame-system has finished. Result: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3051678 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3051678/artifacts/download.

@bkontur
Copy link
Contributor Author

bkontur commented Jun 23, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit dc3fb3d into master Jun 23, 2023
@paritytech-processbot paritytech-processbot bot deleted the bko-set_code-benchmark-helper branch June 23, 2023 12:02
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
paritytech#14435)

* Adds ability to prepare/initialize before running `set_code` benchmark

* Fix

* ".git/.scripts/commands/bench/bench.sh" pallet dev frame-system

* Replaced BenchmarkHelper with function

* Fix

* Introduced `set_code_data` for benchmark with default value

* ".git/.scripts/commands/bench/bench.sh" pallet dev frame-system

* (Hope) Final adjustment (because system parachains generates ValidationFunctionStored instead of CodeUpdated)

* ".git/.scripts/commands/bench/bench.sh" pallet dev frame-system

* ".git/.scripts/commands/bench-vm/bench-vm.sh" pallet dev frame-system

---------

Co-authored-by: command-bot <>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

B0-silent Changes should not be mentioned in any release notes 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.

Fix benchmarks set_code for frame_system pallet for all SP runtimes

4 participants