search

getsentry / sentry-dotnet

Sentry SDK for .NET
https://docs.sentry.io/platforms/dotnet
MIT License
606 stars 206 forks source link

Automatic Heap dump collection #2580

Open bruno-garcia opened 1 year ago

bruno-garcia commented 1 year ago

Specify a memory threshold or other trigger (and API) to automatically capture a heap dump

Sentry attachment preview can be extended to load: https://github.com/1hub/dotnet-heapview/ https://1hub.github.io/dotnet-heapview/

jamescrosswell commented 3 months ago

I think that's what this POC is related to right?

bitsandfoxes commented 2 months ago

Do you have an idea how much work it's going to be to turn the POC into some MVP?

jamescrosswell commented 2 months ago

Do you have an idea how much work it's going to be to turn the POC into some MVP?

A day or two to get what's there productionised.

A little bit more to figure out how to bundle gcdump in our nuget and run it from there.

And a little bit more to do something intelligent with the resulting dump (e.g. send it as an attachment in an issue to Sentry).

About 3 days all up.

bruno-garcia commented 1 month ago

I've fork Filip's repo: https://github.com/getsentry/dotnet-heapview Lets see if we get this reviewed by folks, and we can load the dumps in the attachment preview