jaegertracing / jaeger-operator

Jaeger Operator for Kubernetes simplifies deploying and running Jaeger on Kubernetes.
https://www.jaegertracing.io/docs/latest/operator/
Apache License 2.0
1.02k stars 345 forks source link

[Bug]: `prometheus` storage backend not supported #2244

Closed tsunamaru closed 1 year ago

tsunamaru commented 1 year ago

What happened?

Title. Issue similar to https://github.com/jaegertracing/jaeger-operator/issues/564

Steps to reproduce

  1. Set spec.storage.type to prometheus
  2. k apply -f ...
  3. Watch the logs :)

Expected behavior

Use prometheus as storage backend.

Relevant log output

2023-06-26T11:59:01Z    INFO    The provided storage type is unknown. Falling back to 'memory'  {"instance": "jaeger-dev", "namespace": "jaeger", "storage": "prometheus", "known-options": ["memory","cassandra","elasticsearch","kafka","badger","grpc-plugin"]}

Screenshot

No response

Additional context

Prometheus storage description in jaeger docs: https://www.jaegertracing.io/docs/1.46/cli/#jaeger-all-in-one-prometheus

Jaeger backend version

1.45.0

SDK

No response

Pipeline

No response

Stogage backend

Prometheus

Operating system

Linux

Deployment model

Kubernetes

Deployment configs

apiVersion: jaegertracing.io/v1
kind: Jaeger
metadata:
  name: jaeger-dev
  namespace: jaeger
spec:
  strategy: allInOne
  storage:
    type: prometheus
    options:
      prometheus:
        server-url: http://prometheus.default.svc:8428
iblancasa commented 1 year ago

Hi @tsunamaru. The Prometheus storage is used for the metrics, not for the traces. You need to specify the metrics storage in the .spec.metricsStorage field. There is an example in the examples folder:

apiVersion: jaegertracing.io/v1
kind: "Jaeger"
metadata:
  name: "my-jaeger"
spec:
  strategy: allInOne
  allInOne:
    options:
      log-level: debug
      query:
        base-path: /jaeger
      prometheus:
        server-url: "http://prometheus:9090"
    metricsStorage:
      type: prometheus
  storage:
    options:
      memory:
        max-traces: 100000
tsunamaru commented 1 year ago

@iblancasa thanks for clarification