Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Run omni-bencher overhead cmd in CI #6347

Open
ggwpez opened this issue Nov 4, 2024 · 1 comment
Open

Run omni-bencher overhead cmd in CI #6347

ggwpez opened this issue Nov 4, 2024 · 1 comment
Labels
D0-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder. T10-tests This PR/Issue is related to tests.

Comments

@ggwpez
Copy link
Member

ggwpez commented Nov 4, 2024

We need to ensure the new benchmark overhead command keeps working: #5891
Currently there is a small integration test in the CI, but we should have one like for the pallet benchmarking where we run it for all runtimes in this repo and then lateron also in the fellowship runtimes repo.
@ggwpez ggwpez added T10-tests This PR/Issue is related to tests. D0-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder. labels Nov 4, 2024
@clangenb
Copy link
Contributor

I would like to take on this one. I believe, even if it is a CI change, there is no issue with me not being a maintainer here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
D0-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder. T10-tests This PR/Issue is related to tests.
Projects
Runtime / FRAME
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ggwpez @clangenb