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

Service graphs: add dimensions and rename expired edges metric #1361

Merged
merged 3 commits into from
Apr 6, 2022

Conversation

mapno
Copy link
Member

@mapno mapno commented Apr 4, 2022

Add dimensions to the service graphs processor config, which attaches
additional labels to the metrics from spans attributes.

It also renames the expired edges internal metric.

Copy link
Member

@yvrhdn yvrhdn left a comment

Choose a reason for hiding this comment

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

This will be really neat! Service graphs per cluster or namespace 🤩

Left some comments and questions but looks good overall.

modules/generator/processor/servicegraphs/servicegraphs.go Outdated Show resolved Hide resolved
modules/generator/processor/servicegraphs/servicegraphs.go Outdated Show resolved Hide resolved
pkg/util/attributes.go Outdated Show resolved Hide resolved
cmd/tempo-vulture/main.go Show resolved Hide resolved
Copy link
Member

@yvrhdn yvrhdn left a comment

Choose a reason for hiding this comment

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

Nice work!

mapno added 2 commits April 6, 2022 15:41
Add dimensions to the service graphs processor config, which attaches
additional labels to the metrics from spans attributes.

It also renames the expired edges internal metric.
@mapno mapno force-pushed the servicegraphs-improvements branch from 309793e to 9698163 Compare April 6, 2022 13:58
@mapno mapno enabled auto-merge (squash) April 6, 2022 13:58
@mapno mapno merged commit 9f247d4 into grafana:main Apr 6, 2022
@mapno mapno deleted the servicegraphs-improvements branch April 6, 2022 14:10
@yvrhdn yvrhdn mentioned this pull request Apr 14, 2022
3 tasks
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