Skip to content

Commit

Permalink
[release-1.30] use OTLP exporter instead of the removed jaeger (#2415)
Browse files Browse the repository at this point in the history
* use OTLP exporter instead of the removed jaeger

* use OTLP port on jaeger

* try enabling TLS for OTLP in Jaeger explicitly

---------

Co-authored-by: Marek Schmidt <[email protected]>
  • Loading branch information
openshift-cherrypick-robot and maschmid authored Dec 12, 2023
1 parent 9d38c37 commit 9bed6b4
Showing 1 changed file with 13 additions and 3 deletions.
16 changes: 13 additions & 3 deletions hack/lib/tracing.bash
Original file line number Diff line number Diff line change
Expand Up @@ -196,8 +196,8 @@ spec:
zipkin:
processors:
exporters:
jaeger:
endpoint: jaeger-collector-headless.${TRACING_NAMESPACE}.svc:14250
otlp:
endpoint: jaeger-collector-headless.${TRACING_NAMESPACE}.svc:4317
tls:
ca_file: "/var/run/secrets/kubernetes.io/serviceaccount/service-ca.crt"
logging:
Expand All @@ -206,7 +206,7 @@ spec:
traces:
receivers: [zipkin]
processors: []
exporters: [jaeger, logging]
exporters: [otlp, logging]
EOF
}

Expand All @@ -221,6 +221,16 @@ metadata:
namespace: ${TRACING_NAMESPACE}
spec:
strategy: allInOne
allInOne:
options:
collector:
otlp:
enabled: true
grpc:
tls:
enabled: true
cert: /etc/tls-config/tls.crt
key: /etc/tls-config/tls.key
EOF

logger.info "Wait for Jaeger to be running"
Expand Down

0 comments on commit 9bed6b4

Please sign in to comment.