Skip to content

Add Fuzzer.created_at field#5227

Merged
ViniciustCosta merged 3 commits intomasterfrom
dylanj/fuzzer_created_at
Apr 8, 2026
Merged

Add Fuzzer.created_at field#5227
ViniciustCosta merged 3 commits intomasterfrom
dylanj/fuzzer_created_at

Conversation

@dylanjew
Copy link
Copy Markdown
Collaborator

@dylanjew dylanjew commented Apr 2, 2026

We want to start tracking more metrics around blackbox fuzzers. We currently only track the last updated timestamp of a fuzzer. This introduces a created_at timestamp which is set when creating a fuzzer.

It won't be set for existing fuzzers. If this becomes an important metric to have for all fuzzers, we can manually set this based on other data like Testcases or looking through logs.

The Fuzzer fields are loaded into BigQuery where this metadata will be available for benchmarking

Testing

Ran a local server, created a Fuzzer and verified the field was set in the diffed log.
Added minimal unit test which ensures the created_at field is set and the tests pass

@dylanjew dylanjew force-pushed the dylanj/fuzzer_created_at branch from 89c5af2 to 8229726 Compare April 2, 2026 17:31
@dylanjew dylanjew marked this pull request as ready for review April 2, 2026 17:33
@dylanjew dylanjew requested a review from a team as a code owner April 2, 2026 17:33
@dylanjew dylanjew requested a review from aakallam April 2, 2026 17:33
@dylanjew dylanjew force-pushed the dylanj/fuzzer_created_at branch from 8229726 to 216d643 Compare April 2, 2026 17:34
Copy link
Copy Markdown
Collaborator

@aakallam aakallam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New field LGTM but I have no idea if that's the best way to test this

@dylanjew dylanjew force-pushed the dylanj/fuzzer_created_at branch from e3c125c to 2356b19 Compare April 6, 2026 19:19
dylanjew added 3 commits April 7, 2026 10:11
We want to start tracking more metrics around blackbox fuzzers. We
currently only track the last updated timestamp of a fuzzer. This
introduces a craeted_at timestamp which is set when creating a fuzzer.

Testing:
Ran a local server and creatd a Fuzzer to verify the field is set.
@dylanjew dylanjew force-pushed the dylanj/fuzzer_created_at branch from 2356b19 to 569858c Compare April 7, 2026 14:12
@dylanjew dylanjew requested a review from hunsche April 7, 2026 21:56
Copy link
Copy Markdown
Collaborator

@ViniciustCosta ViniciustCosta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for testing, might be good to add a gpaste or print as evidence next time :)

@ViniciustCosta ViniciustCosta merged commit 870e910 into master Apr 8, 2026
9 of 10 checks passed
@ViniciustCosta ViniciustCosta deleted the dylanj/fuzzer_created_at branch April 8, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants