Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

Bench-Bot v2.0 Specification #8

Open
shawntabrizi opened this issue Nov 3, 2020 · 0 comments
Open

Bench-Bot v2.0 Specification #8

shawntabrizi opened this issue Nov 3, 2020 · 0 comments

Comments

@shawntabrizi
Copy link
Member

shawntabrizi commented Nov 3, 2020

The bench bot was originally created to specifically be used for the node-bench utility. I have, over the last month or so, hacked on top of it the ability to also be used for the runtime benchmarks process.

But these hacks have mostly been... hacky, and some better design and architecture could be used to make the whole tool more robust, easy to upgrade, and more usable.

I suggest the following specification for how users should interact with the bot:

/benchmark (interchangeable with `/bench`)
or
/benchmark help

Return a list of available commands as a Github comment.

/benchmark status

Return availability status and other information about the running processes and queue.

/benchmark specs

Return the machine specifications of the benchmarking device.

/benchmark node <benchmark_filters>

Run a node benchmark using the benchmark filters.

Only whitelisted users should be able to trigger this command.

/benchmark runtime <runtime> <pallet>

Run a runtime benchmark for a specific runtime using the default flags.

runtime options: Substrate, Polkadot, Kusama, Westend, Template(?)

pallet options: Either a specific pallet name or all, which should do some magic to query the available benchmarks on the machine and run them all.

Only whitelisted users should be able to trigger this command.

/benchmark runtime custom <flags>

Run a benchmark with a specific set of flags (thus no need to also specify the runtime).

Only whitelisted users should be able to trigger this command.

/benchmark cancel

Cancel a running benchmark.

Only if you are the person who initiated the benchmark or you are on some whitelist (like Parity Tech Core Devs)

/benchmark cancel all

Clear the entire queue. Only whitelisted users should be able to trigger this command.

/benchmark update

Ideally, update the running process using the latest on the master branch of this repo and restart the bot.

@Vovke Vovke added duplicate This issue or pull request already exists and removed duplicate This issue or pull request already exists labels Apr 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants