-
Notifications
You must be signed in to change notification settings - Fork 740
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.
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
/cmd: Improved devx of benching many pallets simultaneously #6007
Conversation
@@ -19,51 +19,53 @@ In a PR run the actions through comment: | |||
|
|||
To regenerate all weights (however it will take long, | |||
so don't do it unless you really need it), run the following command: | |||
|
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.
This is a great doc, is it properly linked and discover-able from the main CONTRIBUTING.md
?
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.
yep, there is a general link to /cmd commands-readme and from there to here
### Improved devx of running many pallets simultaneously Changes to /cmd: - Replace (flip) `--continue-on-fail` with `--fail-fast`, but only for `bench`. This makes all pallets/runtimes run non-stop by default, as it was primary use-case during tests - The list of successful/failed pallets was hard to find within tons of logs, so decided to write only needed logs in a file, and output as a summary in the workflow and in the comment - Side fix: updated `tasks_example` to `pallet_example_tasks` to make compliant with standard naming <img width="1006" alt="image" src="https://github.com/user-attachments/assets/14896041-7018-4a0d-92b7-4508e81913c2"> + added command results to workflow summary: <img width="1275" alt="image" src="https://github.com/user-attachments/assets/b4a8afdb-dc9f-4ff9-9720-28a88956035f"> --------- Co-authored-by: GitHub Action <[email protected]>
Improved devx of running many pallets simultaneously
Changes to /cmd:
--continue-on-fail
with--fail-fast
, but only forbench
. This makes all pallets/runtimes run non-stop by default, as it was primary use-case during teststasks_example
topallet_example_tasks
to make compliant with standard naming