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

Investigate missing operation from jaeger-query metrics reader #3392

Closed
albertteoh opened this issue Nov 15, 2021 · 2 comments
Closed

Investigate missing operation from jaeger-query metrics reader #3392

albertteoh opened this issue Nov 15, 2021 · 2 comments
Assignees
Labels

Comments

@albertteoh
Copy link
Contributor

Describe the bug

Context: jaegertracing/jaeger-ui#815 (comment)

To Reproduce
Steps to reproduce the behavior:

  1. cd docker-compose/monitor/
  2. docker-compose up
  3. /api/metrics/latencies?service=driver&endTs=1636969517360&groupByOperation=true&lookback=7200000&quantile=0.95&ratePer=3600000&step=60000
  4. (Need to confirm) only receiving metrics for operation /FindNearest.
  5. From main Jaeger UI Search tab, select driver service. Both /FindNearest and HTTP GET operations are present.

Expected behavior
Receive latency metrics for two operations:

  • HTTP GET
  • /FindNearest

Version (please complete the following information):

  • OS: MacOS
  • Jaeger version: master
  • Deployment: dev
@albertteoh albertteoh added the bug label Nov 15, 2021
@albertteoh albertteoh self-assigned this Nov 15, 2021
@Ashmita152
Copy link
Contributor

The prometheus query executed while hitting /api/metrics/latencies has label filter as span_kind =~ "SPAN_KIND_SERVER" while the operation HTTP GET has label span_kind = "SPAN_KIND_CLIENT". Hence it gets filtered out.

@albertteoh
Copy link
Contributor Author

Ah, thanks for investigating, @Ashmita152!

Yes, it's all coming back to me now; we filter on SPAN_KIND_SERVER because we want to aggregate on ingress rather than egress requests (the latter wouldn't make sense IMO).

I'll close this ticket since it's expected (although potentially confusing) behaviour. I'll include this note in the documentation task for ATM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants