Implement fuzzer for Merge_Single

grafana / pyroscope

Continuous Profiling Platform. Debug performance issues down to a single line of code

https://grafana.com/oss/pyroscope/

GNU Affero General Public License v3.0

9.98k stars 599 forks source link

Implement fuzzer for Merge_Single #3222

Closed simonswine closed 5 months ago

simonswine commented 5 months ago

Not too sure if there is a better/more performant way of doing that. But this also uncovered a few cases already 🙂

kolesnikovae commented 5 months ago

I've been running it on main for 2 hours with no failures 🎉

simonswine commented 5 months ago

@aleks-p I have looked into how we can actually run those Fuzz tests regularly. Not too sure if it will work, but at least I tried to have a valid config around 🙂

@kolesnikovae I was curious how often we get post decoding and merge:

postDecode: 5325/522287 (last 3s, total) postMerge 5325/520733 (last 3s, total)

# Also for completeness when encoding/decoding was gob based:
postDecode: 5/39149 (last 3s, total) postMerge 5/39149 (last 3s, total)