@datadog/pprof integration

grafana / pyroscope-nodejs

Pyroscope NodeJS integration

Apache License 2.0

26 stars 18 forks source link

Closed notaphplover closed 4 months ago

notaphplover commented 9 months ago

This is, for now, a work in progress that aims to update the library in such a way it uses the @datadog/pprof library for profiling purposes.

An overview of the changes is the folowing one:

PyroscopeConfig has been updated with equivalent options.
A PyroscopeProfiler has been added internally to encapsulate different API operations.
Pyroscope integration is now enhanced passing from and until parameters to the ingest endpoint.

CLAassistant commented 9 months ago

All committers have signed the CLA.

korniltsev commented 8 months ago

cc @grafakus , @Rperry2174

korniltsev commented 8 months ago

@notaphplover Sorry for long wait. We will take a look at the PR after holidays.

petethepig commented 4 months ago

Hey @notaphplover / @korniltsev — I finished this branch and merged it today.

I had to use the old PR though instead of this one because I could not push to this PR — sorry about the confusion.

And thanks for the contributions!

korniltsev commented 4 months ago

🎉🎉🎉