-
Notifications
You must be signed in to change notification settings - Fork 352
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
add clusterfuzzlite PR fuzzing. #2784
Conversation
b3340ec
to
9befc0e
Compare
Won't this add 10 minutes to PR build? |
it adds about 15 mins, yes, but the check is not mandatory for now. |
the idea is to find bugs during PRs and not wait until the PR is merged and the targets are ran by oss-fuzz. |
@sepehrdaddev that's too long maybe 2m? |
@szuecs I'm not sure which build you are referring to, the one that builds the docker images and etc run in parallel to this (in fact all the checks are running in parallel since this is a separate workflow) as for the timing, it take about 5 mins to build the fuzzers, so I cannot change that but the remaining 10 mins I can change. |
929e12f
to
9bcfe3f
Compare
I have updated the fuzz-seconds to be 300 seconds so its a bit faster. |
Signed-off-by: Sepehrdad Sh <26747519+sepehrdaddev@users.noreply.github.com>
9bcfe3f
to
a2ace44
Compare
👍 |
1 similar comment
👍 |
add clusterfuzzlite PR fuzzing.
it uses the same code as oss-fuzz but it will trigger fuzzing on every PR and it runs for 300 seconds