search

fermitools / managed-tokens

Managed Tokens service for FIFE Experiments at Fermilab
Apache License 2.0
0 stars 0 forks source link

Update trace provider to standard OpenTelemetry HTTP trace provider #96

Closed shreyb closed 1 month ago

shreyb commented 5 months ago

Starting with v0.14, we provide tracing in the Managed Tokens library. This is accomplished using a Jaeger trace provider here.

This decision was made, because currently, the Fermilab monitoring architecture only supports Jaeger traces. Eventually, that will be upgraded to accept more general OpenTelemetry traces. At that point, we should update the function containing the line above to use the OpenTelemetry Trace HTTP exporter package here and update all the dependencies.

I believe, given the construction of the code, that this should be the only function we need to touch.

shreyb commented 2 months ago

Copied from #101:

Our local Fermilab observability stack, Landscape, now has support for OpenTelemetry traces. We should switch the tracing library to use an OpenTelemetry trace exporter, rather than a Jaeger one.

See https://opentelemetry.io/docs/languages/go/instrumentation/
shreyb commented 2 months ago

This is now available to do.

shreyb commented 1 month ago

Done. Will open new PR next week.