-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
util/db: add fuzzer #11146
util/db: add fuzzer #11146
Conversation
Signed-off-by: AdamKorcz <adam@adalogics.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #11146 +/- ##
==========================================
- Coverage 48.99% 48.99% -0.01%
==========================================
Files 246 246
Lines 42486 42485 -1
==========================================
- Hits 20816 20815 -1
Misses 19559 19559
Partials 2111 2111 ☔ View full report in Codecov by Sentry. |
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.
Nice, thanks @AdamKorcz! Can you run go mod tidy
and push?
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@AdamKorcz mind also fixing DCO? Once I merge, what how do we actually use this fuzzer? |
This seems to be using Go 1.18+ built-in fuzzing, which can be run with Usually they're run continuously via OSS-Fuzz as well as for a short period of time on PRs. The CNCF fuzzing repo actually describes the same thing in more detail. It seems like Argo was already added to OSS-Fuzz at some point, although I'm not entirely sure where the fuzz tests are referenced (especially as most of them are located in the above mentioned CNCF fuzzing repo) |
Closed the PR due to it being old and unlikely to be able to resolve merge conflicts. But feel free to re-open/re-submit if the work is still planned. |
Moves a fuzzer from https://github.com/cncf/cncf-fuzzing/blob/main/projects/argo/argo-cd_db_fuzzer.go
Signed-off-by: AdamKorcz adam@adalogics.com
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist: