Skip to content

Conversation

ndr-ds
Copy link
Contributor

@ndr-ds ndr-ds commented Sep 2, 2025

Motivation

We're adding Pyroscope support back, but using eBPF this time

Proposal

  • Add Pyroscope to our helm chart
  • Add a Grafana dashboard with the profiling info

Test Plan

Deployed a network and saw the Pyroscope profiling data both on the Pyroscope UI and on Grafana, as well as logs from Pyroscope on Loki:

Screenshot 2025-09-02 at 19.31.14.png

Screenshot 2025-09-02 at 19.31.04.png

Screenshot 2025-09-02 at 19.30.55.png

Release Plan

  • Nothing to do / These changes follow the usual release cycle.

Copy link
Contributor Author

ndr-ds commented Sep 2, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ndr-ds ndr-ds requested review from afck, deuszx, eldios, ma2bd and Twey September 2, 2025 23:38
Copy link
Contributor

@deuszx deuszx left a comment

Choose a reason for hiding this comment

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

I always find it difficult to review graphana dashboards – if it works (for you) then I approve :)

@ndr-ds ndr-ds force-pushed the 09-02-ebpf_pyroscope branch from 03025d4 to 7685773 Compare September 8, 2025 20:47
@ndr-ds ndr-ds added this pull request to the merge queue Sep 9, 2025
Merged via the queue into main with commit 240e7a9 Sep 9, 2025
29 checks passed
@ndr-ds ndr-ds deleted the 09-02-ebpf_pyroscope branch September 9, 2025 05:51
ndr-ds added a commit that referenced this pull request Sep 22, 2025
## Motivation

We're adding Pyroscope support back, but using eBPF this time

## Proposal

* Add Pyroscope to our helm chart
* Add a Grafana dashboard with the profiling info

## Test Plan

Deployed a network and saw the Pyroscope profiling data both on the
Pyroscope UI and on Grafana, as well as logs from Pyroscope on Loki:

![Screenshot 2025-09-02 at
19.31.14.png](https://app.graphite.dev/user-attachments/assets/e02b7de5-3359-462c-8eec-f4ccf7b46456.png)

![Screenshot 2025-09-02 at
19.31.04.png](https://app.graphite.dev/user-attachments/assets/cbbd7e08-34c3-4152-bd77-68525734021a.png)

![Screenshot 2025-09-02 at
19.30.55.png](https://app.graphite.dev/user-attachments/assets/00435910-5842-4c09-b4aa-634c6e51eace.png)

## Release Plan

- Nothing to do / These changes follow the usual release cycle.
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.

2 participants