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

Add statsd monitor documentation #2996

Merged
merged 5 commits into from
Feb 21, 2025
Merged

Conversation

Evian-Zhang
Copy link
Contributor

Currently, this is a Draft PR. I will finish this asap. There have been several busy weeks for me, so I may be slow to respond and update.

@domenukk
Copy link
Member

Already looks good, we can merge this for now if you want?

@Evian-Zhang
Copy link
Contributor Author

There are still plenty of key contents missing. A grafana dashboard template should be added to help visualizing the data, and I will add a screenshot and several FAQs sooner. Maybe a few days later I could complete this. (Life is hard for a PhD candidate😂)

@domenukk
Copy link
Member

Thanks for doing it in any case! Happy to merge this whenever you feel like it

@Evian-Zhang Evian-Zhang marked this pull request as ready for review February 19, 2025 13:50
@Evian-Zhang
Copy link
Contributor Author

I have completed the doc for StatsdMonitor. Please feel free to review this PR. :)

@tokatoka
Copy link
Member

why libfuzzer_stb_image_sugar always fails at your branch 😂
yes the script is bad for false positives but it's very rare for us to hit that failure

@Evian-Zhang
Copy link
Contributor Author

Maybe GitHub uses different runner machines based on user location? It's indeed weird, though😆

@Evian-Zhang
Copy link
Contributor Author

I think the CI checks are all passed. If there are still something to improve, feel free to review my PR :)

@tokatoka tokatoka merged commit 1debeb7 into AFLplusplus:main Feb 21, 2025
107 checks passed
@domenukk
Copy link
Member

Thank you!

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