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

Add user and org labels to observed exemplars #237

Merged
merged 1 commit into from
Apr 28, 2022

Conversation

colega
Copy link
Contributor

@colega colega commented Feb 15, 2022

Exemplars provide information about traces, and traces usually reference to the user that served the request, however, searching the user in the trace is usually a tedious task, and it would be nice to be able to overwiew whether the slow requests (exemplars) from your panel corresponds to the same user.

We could even consider adding exemplars from non-traced requests that have a user ID? This would provide (poor) insights about the request patterns even for the users who run the server without tracing set up.

Exemplars provide information about traces, and traces usually reference
to the user that served the request, however, searching the user in the
trace is usually a tedious task, and it would be nice to be able to
overwiew whether the slow requests (exemplars) from your panel
corresponds to the same user.

Signed-off-by: Oleg Zaytsev <[email protected]>
@bboreham
Copy link
Collaborator

I agree the caller ID(s) would be useful when there is no trace ID.
However I can also see some downsides:

  • current implementation only sends one exemplar per series per scrape, so that change stands the chance of overwriting an exemplar with a trace ID with a different exemplar with no trace ID.
  • it might show more overhead.

@colega
Copy link
Contributor Author

colega commented Apr 28, 2022

Yes, I agree especially on the first point, for people who don't sample all requests that would be a step back.

Can we merge this?

@bboreham bboreham merged commit f83ccc7 into weaveworks:master Apr 28, 2022
@colega colega deleted the user-and-org-labels-on-exemplars branch July 19, 2022 09:55
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