-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
CI: Really run fuzzer for 5 minutes #1338
Conversation
NOTE: I'm doing improvements. |
Funny thing is that this PR is blocked by #1335 :( |
The What would you think could be a good duration for this CI? 2 minutes is obviously very little. |
I think 2 minutes would detect most of the issues indeed. |
Really? I thought 2 minutes is too little... |
Let me please increase it to 5 minutes. |
@ibc, in order to run the fuzzer this must be run:
The |
I've read the comments in meson PR. So those args should be added in fuzzer_run_all task in worker/tasks.py right? |
Working fine. Merging. |
Run
invoke fuzzer-run-all
and wait for 5 minutes. Then check if it was still running. If not it means that it existed due to some (fuzzer) error so CI would fail.NOTE: This was not needed finally but just in case, the fuzzer binary exists with status 77 if a fuzzer issue happened and with status 72 (internally documented) if interrupted via
SIGINT
. Docs: https://llvm.org/docs/LibFuzzer.htmlBonus Track: Fix #1334.
Bonus Track: Fix #1335.