ci: merge seed corpora with public OSS-Fuzz corpora on CFLite #22302
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
evverx
force-pushed
the
merge-oss-fuzz-corpora
branch
2 times, most recently
from
21d58e4
to
a6046e5
Compare
|
Judging by https://github.com/systemd/systemd/actions/runs/1767066906 it seems to be working. For example the fuzz-varlink seed corpus was merged with the OSS-Fuzz corpus and the fuzzer used 5088 files (instead of 10) |
evverx
force-pushed
the
merge-oss-fuzz-corpora
branch
from
a6046e5
to
7237edb
Compare
|
It's blocked by google/clusterfuzzlite#84 :-( |
to let it cover as much code as possible. It should help to catch most regressions in a few minutes.
The idea behind this action is to make it possible to compare the latest fuzz targets with PRs to figure out whether bugs are really reproducible in PRs only. Since forks (including systemd-stable) are usually based on the upstream repository where almost all the bugs are fixed before releases are cut it should be safe to assume that if CFLite finds bugs in PRs they are most likely introduced in those PRs. It should probably be brought back once google/clusterfuzzlite#84 is fixed.
evverx
force-pushed
the
merge-oss-fuzz-corpora
branch
from
7237edb
to
a75f166
Compare
|
@mrc0mmand to get it around I decided to no longer upload artifacts containing the latest build The action is being tested in https://github.com/evverx/systemd/runs/4997912999?check_suite_focus=true and once it's green it should be good to go I think. |
mrc0mmand
reviewed
Jan 30, 2022
mrc0mmand
approved these changes
Jan 30, 2022
|
Looks like CI is happy with this change (given the job you've posted), so LGTM. |
evverx
added
the
reviewed/needs-rework 🔨
evverx
force-pushed
the
merge-oss-fuzz-corpora
branch
from
a75f166
to
d38363b
Compare
evverx
removed
the
reviewed/needs-rework 🔨
|
Still LGTM, thanks! Let's wait for the job to finish. |
mrc0mmand
added
the
good-to-merge/waiting-for-ci 👍
This was referenced Mar 31, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
to let it cover as much code as possible. It should help to catch most regressions in a few minutes.