Tracing pipeline dashboard and alerts - #1166
Conversation
mapno
changed the title
rfratto
left a comment
rfratto
left a comment
There was a problem hiding this comment.
LGTM from jsonnet perspective, but I didn't validate the dashboards or alerts since you're the expert there
annanay25
left a comment
annanay25
left a comment
There was a problem hiding this comment.
Great work on adding the agent tracing mixin!
Question: Wondering if some of the info on the panels is redundant. Do we need individual panels for Received, Accepted and Rejected spans? Similarly for exported spans.
Sure, it's kind of the same information but displayed in different ways. The panels in the first row give per-pod details, while the other one give a more general view. Does that make sense? |
|
Ok, might be good to add a legend |
It's added, but it's hidden because it could become too cluttered in big deployments. Similar to Tempo's operational dashboard which does the same (you can see where I took inspiration from :)) |
* Jsonnet lint * Upgrade jsonnet libs * Tracing pipeline dashboard * Changelog * Lint * Lint * Specify transport for receivers * Add alerts * Changelog * Add grafana prefix * Fix jsonnet formatting
github-actions
Bot
added
the
frozen-due-to-age
3 participants
PR Description
Adds new
Agent Tracing Pipelinejsonnet dashboard and prometheus alerts to production mixin.Other changes:
PR Checklist