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

Implement Pyroscope reporting #555

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

danog
Copy link

@danog danog commented Feb 21, 2023

Based on #554, implements Pyroscope reporting directly inside py-spy.

@danog
Copy link
Author

danog commented Mar 27, 2023

Ping? :)

Copy link
Owner

@benfred benfred left a comment

Choose a reason for hiding this comment

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

This might be more of a questions for @petethepig - but I'm wondering where this PR fits in with the pyroscope roadmap.

I saw that pyroscope was recently acquired by Grafana recently (congratulations on this btw!), and I'm wondering what the plans are moving forward for using py-spy with pyroscope. There used to be a pyroscope project https://github.com/pyroscope-io/py-spy for integration - but I see that project is now archived.

@xtrime-ru xtrime-ru force-pushed the pyroscope_pr branch 18 times, most recently from a8e6d36 to b443050 Compare May 7, 2024 16:26
@xtrime-ru xtrime-ru force-pushed the pyroscope_pr branch 5 times, most recently from 7a438b3 to ccfdc59 Compare May 7, 2024 16:55
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