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

Process metrics dashboard #1109

Merged
merged 5 commits into from
Sep 12, 2024
Merged

Process metrics dashboard #1109

merged 5 commits into from
Sep 12, 2024

Conversation

mariomac
Copy link
Contributor

@mariomac mariomac commented Aug 27, 2024

It works with both OTEL-generated metrics (which sends resource attributes separated in the target_info metric) and Prometheus-scraped metrics (which send resource attributes in the same metric). The screenshot shows the graphs split in two parts: the left part is capturing the Prometheus metrics, and the right part is after switching Beyla to export via OTEL endpoint.

It achieves that by having two types of queries:

  • OTEL: A query of the process metric selected with process_pid="" (as the OTEL exporter does not send this attribute in the metric), joined with target_info{process_pid!=""} metric
  • Prometheus: a query of the process metric selected with process_pid!="" (as the Prometheus exporter includes this attribute in the metric)

So, to let this dashboard work, the user should let the process_pid attribute enabled (default).

image

@codecov-commenter
Copy link

codecov-commenter commented Aug 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.20%. Comparing base (487a85a) to head (4aed8fd).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1109      +/-   ##
==========================================
- Coverage   81.96%   79.20%   -2.76%     
==========================================
  Files         141      136       -5     
  Lines       11595    11293     -302     
==========================================
- Hits         9504     8945     -559     
- Misses       1563     1803     +240     
- Partials      528      545      +17     
Flag Coverage Δ
integration-test ?
k8s-integration-test 59.47% <ø> (+0.39%) ⬆️
oats-test 36.01% <ø> (-0.84%) ⬇️
unittests 53.81% <ø> (+1.37%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

{
"datasource": {
"type": "prometheus",
"uid": "${DS_GRAFANACLOUD-MARIOMACIAS-PROM}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we want to rename this something more generic like ${datasource}

@@ -0,0 +1,556 @@
{
"__inputs": [
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't see this section in our other dashboards, so maybe is not needed?

Copy link
Contributor Author

@mariomac mariomac Sep 2, 2024

Choose a reason for hiding this comment

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

Yeah. This dashboard is exported with the "Export for sharing externally" option, while the others don't. I'm not sure which is the correct form. I tried exporting/importing in both formats, and the difference is that, during the import, this form allows choosing the source in the import dialog.

@mariomac mariomac changed the title WIP: Process metrics dashboard Process metrics dashboard Sep 10, 2024
@mariomac mariomac removed the wip label Sep 10, 2024
@mariomac mariomac merged commit 1c46636 into grafana:main Sep 12, 2024
5 of 6 checks passed
@mariomac mariomac deleted the proc-dash branch September 12, 2024 08:32
mattdurham pushed a commit to mattdurham/beyla that referenced this pull request Jan 22, 2025
* First snapshot of process metrics dashboard

* make it work with prometheus-scraped metrics

* * Addressed Marc's comments
* Divided disk and network panels in send/receive
* Reformat legend

* revert unwanted changes in dashboard.json
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