search

open-telemetry / opentelemetry-erlang-api

Erlang/Elixir OpenTelemetry API
Apache License 2.0
60 stars 13 forks source link

HTTP Integration #31

Closed tsloughter closed 3 years ago

tsloughter commented 4 years ago

For beta there must be at least 1 HTTP integration available.

This will be a separate repo and likely https://github.com/opentelemetry-beam/opentelemetry_elli or https://github.com/opentelemetry-beam/opentelemetry_plug but neither is complete yet so tracking this here.

hauleth commented 4 years ago

I can handle Plug integration, however I think we should integrate with Telemetry as well as that would provide integrations for more services.

tsloughter commented 4 years ago

@hauleth any update on this? I'd forgotten you said you would handle this, I've been bringing it up in the slack channel as I've been thinking about how best to approach it. Doing it through Telemetry is probably best, but it likely still needs a separate library of its own.

I'm planning a Tesla integration and it will require its own lib that also uses Telemetry integrations.

Hm, or for these simple ones like Tesla we could have the modules just be part of opentelemetry_telemetry, instead of a bunch of separate libs that are just 1 module.

hauleth commented 4 years ago

I was thinking about opentelemetry_telemetry library. I need to sit and do so, I will try to provide PoC this weekend.