jaegertracing / jaeger-ui

Web UI for Jaeger
http://jaegertracing.io/
Apache License 2.0
1.11k stars 475 forks source link

[Bug]: Special cases are not handled correctly when using an elasticsearch backend with ES_TAGS_AS_FIELDS_ALL set #989

Open Limess opened 1 year ago

Limess commented 1 year ago

What happened?

As a Jaeger user with Elasticsearch as a backend I want the Jaeger UI to respect ES_TAGS_AS_FIELDS_DOT_REPLACEMENT So I can use sophisticated UI functionality while using Elasticsearch as a backend

Currently, https://github.com/jaegertracing/jaeger-ui/pull/659 is not working due to peer.service becoming peer_service (similar with other special functionality/fields implemented here: https://github.com/jaegertracing/jaeger-ui/blob/e197d1c8d36a73616620c2b4d55d18f4a017a188/packages/jaeger-ui/src/model/trace-dag/denseTransforms.tsx

Steps to reproduce

Expected behavior

I'd expect the PeerService to be shown as per this PR https://github.com/jaegertracing/jaeger-ui/pull/659

Relevant log output

No response

Screenshot

peer.service is converted to peer_service, so the code path is not triggered. image

Additional context

No response

Jaeger backend version

v1.37.0

SDK

OpenTelemetry Java Agent v1.17.0

Pipeline

OTEL SDK -> OTEL Collector -> Jaeger Collector -> [Jaeger Proto] -> Elasticsearch (AWS Opensearch 1.3)

Stogage backend

AWS Opensearch 1.3

Operating system

Linux

Deployment model

Docker ECS Daemons

Deployment configs

ES_VERSION: "7"
      ES_SERVER_URLS: <es_backend_url>
      ES_TAGS_AS_FIELDS_ALL: "true"
      ES_TAGS_AS_FIELDS_DOT_REPLACEMENT: _
      SPAN_STORAGE_TYPE: elasticsearch
Limess commented 1 year ago

This only seems to occur when using a custom label, e.g _ rather than the default @. We've now switched to the default so are less interested in the resolution of this.