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

[CI/CD] Add benchmarking CI job #631

Closed
apopiak opened this issue Sep 29, 2021 · 2 comments · Fixed by #1074
Closed

[CI/CD] Add benchmarking CI job #631

apopiak opened this issue Sep 29, 2021 · 2 comments · Fixed by #1074
Labels
T7-system_parachains This PR/Issue is related to System Parachains.

Comments

@apopiak
Copy link
Contributor

apopiak commented Sep 29, 2021

It would be great to have a manual job to execute runtime benchmarks we can start from gitlab that spits out a git patch of the adjusted weights we can apply afterwards.
Analogous to the benchmarking CI in the Polkadot repo.

@NachoPal
Copy link
Contributor

Is it work in progress? I need to update the Benchmark section in #661 and I am not sure if this is going to be done soon or we keep the manual process for now.

@NachoPal NachoPal added the T7-system_parachains This PR/Issue is related to System Parachains. label Oct 15, 2021
@sea212
Copy link

sea212 commented Nov 17, 2021

Maybe https://github.com/zeitgeistpm/zeitgeist/blob/main/.github/workflows/benchmark.yml can become handy.
Once "benchmark-required" label is set on a PR, the CI is triggered and pulls the branch. It benchmarks currently every pallet and the pallets are hard-coded, but this will be optimized in the future. A report is created and posted in the PR and the changes are committed, if no other changes have been added meanwhile. The CI indicates with a label that it's done and whether the benchmark was successful.
The following link references setting the label and the resulting actions from the benchmark (expand following comments): zeitgeistpm/zeitgeist#207 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T7-system_parachains This PR/Issue is related to System Parachains.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants