-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
chore: fix stress test #3233
chore: fix stress test #3233
Conversation
CI run 1515308234 reported that the command Run apt install -y valgrind failed due to missing root permissions. This patch adds sudo before the apt install to overcome this.
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 workflow is not enabled for PR.
Please add pull_request
condition to on
key or move stress-test
job to ci.yml
.
tokio/.github/workflows/stress-test.yml
Lines 2 to 5 in e013913
on: | |
push: | |
branches: | |
- master |
on:
+ pull_request:
push:
branches:
- master
Co-authored-by: Taiki Endo <te316e89@gmail.com>
Co-authored-by: Taiki Endo <te316e89@gmail.com>
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.
LGTM, thanks for the fix!
thanks for the great feedback! |
CI run 1515308234 reported that the command
Run apt install -y valgrind failed due to missing root permissions.
This patch adds sudo before the apt install to overcome this.
Motivation
Solution