Skip to content

Add old state and diff logging on fuzzer update#5184

Merged
decoNR merged 9 commits intogoogle:masterfrom
tbantikyan:fuzzer-update-logging
Mar 12, 2026
Merged

Add old state and diff logging on fuzzer update#5184
decoNR merged 9 commits intogoogle:masterfrom
tbantikyan:fuzzer-update-logging

Conversation

@tbantikyan
Copy link
Contributor

Logs currently include only the fuzzer name. Previous state and diff information can help identify issues with any updates to fuzzers.

@tbantikyan tbantikyan requested a review from a team as a code owner March 3, 2026 14:48
Copy link
Collaborator

@letitz letitz left a comment

Choose a reason for hiding this comment

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

LGTM to avoid blocking on me further, modulo a couple comments below.

Copy link
Collaborator

@javanlacerda javanlacerda left a comment

Choose a reason for hiding this comment

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

Could you add unit tests for this change?

@tbantikyan
Copy link
Contributor Author

Could you add unit tests for this change?

Thank you for the suggestion. Added them in: a73fc33

@decoNR
Copy link
Contributor

decoNR commented Mar 10, 2026

/gcbrun

@decoNR decoNR merged commit 9f0c774 into google:master Mar 12, 2026
11 checks passed
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.

4 participants