FuelLabs / fuel-vm

Fuel v2 interpreter in Rust
Other
358 stars 88 forks source link

Add ClusterFuzzLite in CI featuring PR fuzzing, batch fuzzing and fuzz coverage reports #820

Closed netrome closed 1 month ago

netrome commented 2 months ago

Supersedes https://github.com/FuelLabs/fuel-vm/pull/727

This PR introduces fuzzing in our CI using ClusterFuzzLite, roughly following the instructions outlined in the ClusterFuzzLite Github Actions docs.

Before requesting review

netrome commented 1 month ago

It would be nice to verify somehow that if fuzzers fails, it will report about it. Could you try to introduce a bug in a separate PR which fuzzer will catch, please?=)

We had some fun doing this in this PR. Posting here for visibility: https://github.com/FuelLabs/fuel-vm/pull/846

maxammann commented 1 month ago

@netrome Hi! Would you be open to chat about how this works our for Fuel in a few months from now?

netrome commented 1 month ago

@netrome Hi! Would you be open to chat about how this works our for Fuel in a few months from now?

Sure thing, I'd be happy to. Right now we're experiencing some issues with empty coverage uploads https://github.com/FuelLabs/fuel-fuzzing-corpus/commit/bf288985f3b94e8048d36b457c750cf7af6865b5 is that anything you recognize from other projects?

maxammann commented 1 month ago

@netrome Hi! Would you be open to chat about how this works our for Fuel in a few months from now?

Sure thing, I'd be happy to. Right now we're experiencing some issues with empty coverage uploads FuelLabs/fuel-fuzzing-corpus@bf28898 is that anything you recognize from other projects?

Sadly does not ring a bell for me :/

netrome commented 1 month ago

Sadly does not ring a bell for me :/

Oh well, we'll figure it out.