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

Initial fuzzing tests #572

Merged
merged 2 commits into from
Feb 9, 2022
Merged

Initial fuzzing tests #572

merged 2 commits into from
Feb 9, 2022

Conversation

pjbgf
Copy link
Member

@pjbgf pjbgf commented Feb 8, 2022

Following-up from the security audit, we reviewed the initial Fuzz PR and created this PR that supersedes it.

Main changes:

  • Structure the fuzz implementation to be closer to what fuzz testing might look like once leveraging the golang 1.18 built-in fuzz feature.
  • Add Makefile target to enable smoke testing fuzzers locally.
  • Add github workflow to smoke test fuzzers.

Co-authored-by: @aryan9600 @AdamKorcz

@pjbgf pjbgf marked this pull request as draft February 8, 2022 16:00
@pjbgf pjbgf force-pushed the new-fuzz branch 2 times, most recently from 14f98ce to 8e8a24c Compare February 8, 2022 16:41
Paulo Gomes and others added 2 commits February 8, 2022 17:49
This PR refactors and supersedes: fluxcd#443

Signed-off-by: Paulo Gomes <paulo.gomes@weave.works>
Co-authored-by: Sanskar Jaiswal <sanskar.jaiswal@weave.works>
Co-authored-by: AdamKorcz <adam@adalogics.com>
Signed-off-by: Paulo Gomes <paulo.gomes@weave.works>
@pjbgf pjbgf marked this pull request as ready for review February 8, 2022 18:42
@hiddeco hiddeco added the area/ci CI related issues and pull requests label Feb 9, 2022
@hiddeco hiddeco merged commit 657f80b into fluxcd:main Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants