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.

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: add CIFuzz github action #283

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DavidKorczynski
Copy link

Add CIFuzz workflow action to have fuzzers build and run on each PR.

This is a service offered by OSS-Fuzz where reSIProcate already runs. CIFuzz can help detect regressions and catch fuzzing build issues early, and has a variety of features (see the URL above). In the current PR the fuzzers gets build on a pull request and will run for 300 seconds.

@gjasny
Copy link
Contributor

gjasny commented Nov 23, 2022

  • I'd like to check if the build time of 11 minutes could be shortened

DavidKorczynski added a commit to google/oss-fuzz that referenced this pull request Nov 23, 2022
Avoid building libfmt from source and just use a package. This is particularly needed to make CIFuzz faster: resiprocate/resiprocate#283 (comment)
AdamKorcz pushed a commit to google/oss-fuzz that referenced this pull request Nov 23, 2022
Avoid building libfmt from source and just use a package. This is
particularly needed to make CIFuzz faster:
resiprocate/resiprocate#283 (comment)
@DavidKorczynski
Copy link
Author

DavidKorczynski commented Dec 9, 2022

@gjasny I reduced the fuzzer run time and also the build time -- let me know if this is better suited

@gjasny
Copy link
Contributor

gjasny commented Dec 18, 2022

Once google/oss-fuzz#9232 has been merged, could you please rebase?

eamonnmcmanus pushed a commit to eamonnmcmanus/oss-fuzz that referenced this pull request Mar 15, 2023
Avoid building libfmt from source and just use a package. This is
particularly needed to make CIFuzz faster:
resiprocate/resiprocate#283 (comment)
@DavidKorczynski
Copy link
Author

@gjasny -- apologies for the delay here. I rebased it now.

Signed-off-by: David Korczynski <david@adalogics.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants