search

grafana / helm-charts

Apache License 2.0
1.59k stars 2.24k forks source link

[tempo] missing port 4318 in tempo stateful set #2282

Open yosefy opened 1 year ago

yosefy commented 1 year ago 
    - containerPort: 4317
      name: otlp-grpc
    - containerPort: 55681
      name: otlp-http

but 55681 is really legacy port and service forwards this port to target port 4318

it will not work like this please add

        - containerPort: 55681
          name: legacy-otlp-http
        - containerPort: 4318
          name:otlp-http

to https://github.com/grafana/helm-charts/blob/main/charts/tempo/templates/statefulset.yaml

replacing

        - containerPort: 55681
          name: otlp-http

thanks

Sheikh-Abubaker commented 10 months ago

Hi @yosefy is this issue fixed ? as I can see in https://github.com/grafana/helm-charts/blob/main/charts/tempo/templates/statefulset.yaml port 4318 is present