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

refactor: Replace @pyroscope/flamegraph dependency #1870

Open
3 tasks
yurishkuro opened this issue Oct 14, 2023 · 3 comments
Open
3 tasks

refactor: Replace @pyroscope/flamegraph dependency #1870

yurishkuro opened this issue Oct 14, 2023 · 3 comments

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Oct 14, 2023

The company behind this module has been acquired by Grafana. The repo moved there https://github.com/grafana/pyroscope, and they changed the license to AGPL, to match the rest of Grafana. AGPL is not allowed in CNCF projects.

We're currently on an old version, and no new versions are expected. It's not an issue right now, but eventually there may be security issues, if not in that module than in its dependencies.

  • research possible replacements, present pros / cons
  • make a decision with maintainers which replacement to use
  • make code changes to replace the library
@RipulHandoo
Copy link

Hi @yurishkuro,

I noticed the use of Pyroscope in our codebase here and the corresponding test case here.

Considering Pyroscope's move to Grafana with AGPL licensing, I suggest exploring pprof as a potential replacement for our Jaeger UI project.

Steps:

  1. Research pprof's suitability.
  2. Discuss with maintainers.
  3. Decide and implement changes.

Ensuring project security and stability is crucial, and I believe pprof could align well with our needs.

@yurishkuro
Copy link
Member Author

pprof is a standalone tool, not a viz library.

@EshaanAgg
Copy link
Contributor

EshaanAgg commented Dec 26, 2023

react-flame-graph seems to be the most popular flame-graph visualisation library out there. It is developed by Brian Vaughn, a core React developer who amoung other things also worked on introducing the React Profiler. It seems like a great library that can be said to be the community's "go-to choice", but it hasn't received any updates since 4 years ago.

Also I couldn't find any other tools that allowed the direct conversion of a Jaegar trace to a flame graph, which could pose a problem for us. As a workaround, can we include this script for conversion from pyroscope/flamegraph into our codebase directly with proper attribution in a way that doesn't violate the licensing requirements?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants