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

[Bug]: NoData for Latency and ErrorRate in Monitoring tab with Jaeger 1.46 and Opentelemetry SpanMetrics Connector #4502

Closed
Tyrion85 opened this issue Jun 5, 2023 · 8 comments
Assignees

Comments

@Tyrion85
Copy link

Tyrion85 commented Jun 5, 2023

What happened?

This issue added support for SpanMetrics connector (since SpanMetrics Processor is deprecated)

However, I'm not seeing latency and error rate in Jaeger's Monitoring tab - only Request rate is populated, and a "No data yet!" notification is displayed in UI.

Relevant metrics are written to Prometheus by OpenTelemetry Collector, as shown in the screenshots below. Jaeger metrics jaeger_query_requests_total{operation="get_latencies",result="ok"} and similar suggest everything is okay. There is also no error visible in Query logs.

Jaeger version 1.46.0, installed via helm chart (not a all-in-one)

Steps to reproduce

  1. Install Jaeger via helm chart, image version 1.46.0
  2. Setup OpenTelemetry collector with config from upstream
  3. Connect Jaeger with Prometheus (in my case, Thanos, but it doesn't matter)
  4. Inspect that Latency and Error Rate are missing

Expected behavior

Expected Latency and Error Rate to work, as related metrics are available in Prometheus

Relevant log output

No response

Screenshot

Screenshot 2023-06-05 at 15 53 53 Screenshot 2023-06-05 at 15 49 23 Screenshot 2023-06-05 at 15 50 49 Screenshot 2023-06-05 at 15 53 07 Screenshot 2023-06-05 at 15 49 39

Additional context

Relevant OpenTelemetry Collector configuration:

receivers:
      otlp:
        protocols:
          grpc:
          http:
          ....
      zipkin:
      opencensus:
      jaeger:
        protocols:
          grpc:
          thrift_http:
          thrift_compact:
          thrift_binary:
    connectors:
      spanmetrics:
    processors:
      batch:
      span/to_attributes:
        ...
    extensions:
      health_check:
    exporters:
      jaeger:
        ....
      otlp/quickwit:
        ....
      prometheusremotewrite:
        endpoint: http://prometheus-operated.monitoring:9090/api/v1/write
        target_info:
          enabled: true
    service:
      extensions: [ health_check ]
      pipelines:
        traces:
          receivers: [ otlp,opencensus,zipkin,jaeger ]
          processors: [ batch,span/to_attributes ]
          exporters: [ jaeger,otlp/quickwit,spanmetrics ]
        metrics:
          receivers: [ spanmetrics ]
          exporters: [ prometheusremotewrite ]

Relevant Jaeger config (helm chart):

....
extraEnv:
    - name: GRPC_STORAGE_SERVER
      value: quickwit-searcher:7281
    - name: METRICS_STORAGE_TYPE
      value: prometheus
    - name: PROMETHEUS_SERVER_URL
      value: http://thanos-query-frontend.monitoring:9090

Jaeger backend version

v1.46.0

SDK

No response

Pipeline

No response

Stogage backend

GRPC_STORAGE_SERVER (Quickwit)

Operating system

No response

Deployment model

Kubernetes, helm chart

Deployment configs

No response

@Tyrion85 Tyrion85 added the bug label Jun 5, 2023
@Tyrion85
Copy link
Author

Tyrion85 commented Jun 5, 2023

--prometheus.query.support-spanmetrics-connector=true command line argument was missing from the config. This is not documented in Jaeger setup docs (it is in Migration docs, but you'd have to be aware that some kind of migration was going on). https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md#migrating

Closing this issue & leaving the comment if someone else stumbles on the same

@Tyrion85 Tyrion85 closed this as completed Jun 5, 2023
@yurishkuro
Copy link
Member

command line argument was missing from the config. This is not documented in Jaeger setup docs

Which docs are you referring to, the README in the repo or the website?

Re-opening to track a need to update the docs.

@Tyrion85
Copy link
Author

Tyrion85 commented Jun 5, 2023

@Tyrion85
Copy link
Author

Tyrion85 commented Jun 5, 2023

Should probably be somewhere here:

Screenshot 2023-06-05 at 17 01 26

@yurishkuro
Copy link
Member

Thanks! Added to both places.

@hdong69
Copy link

hdong69 commented Jun 6, 2023

Anyone can confirm that the Jaeger 1.46 works well with the tutorial https://github.com/jaegertracing/jaeger/blob/main/docker-compose/monitor/README.md#migrating ?

@Tyrion85
Copy link
Author

Tyrion85 commented Jun 6, 2023

It works well for me, but I was not migrating from OTEL SpanMetrics Processor config, I went straight into SpanMetrics Connector. Configuration was pretty simple except for the missing flag prometheus.query.support-spanmetrics-connector from the main docs page, which is now amended 👍🏼

@hdong69
Copy link

hdong69 commented Jun 6, 2023

@Tyrion85 thanks for your feedback

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

No branches or pull requests

4 participants