You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Microservice architecture is often required integrated observability.
Open-Telemetry is attracted attention one of the integrated observability method.
hello @aldas , would it make sense to move Open Telemetry support into this repo from open-telemetry/opentelemetry-go-contrib? I've opened a PR in there to support echo v5 (open-telemetry/opentelemetry-go-contrib#8457), but there is a question about the ownership and the actual location of the new package.
What do you think? I'd be happy to open a PR against this repo
@vadimi Hi, I'm probably a bit of a Luddite by today's standards, and I haven't used Open Telemetry before. That middleware part config.go+echo.go does not seem too complex. so we probably can support it. If you look at the commit history this repo is not that active, so I do not know if this suitable for otel tempo?
about internal/semconv stuff in that PR - without me trying to understand that - is that also needed?
internal/semconv is normally used for keeping otel attributes that will be used in spans/traces, it's autogenerated in open-telemetry repo, but it doesn't necessary need to be structured like that here.
In terms of activity - I don't expect lots of changes and we can keep it as a sub-package to avoid pulling too many dependencies in the main echo-contrib module similar to how it's done in open-telemetry contrib repo or https://github.com/redis/go-redis/tree/master/extra/redisotel
One thing I'm not sure about is github.com/labstack/echo-contrib/otelecho/v5 path for the sub module to be able to introduce breaking changes into echo, but keep the previous versions still available. Please let me know if you're ok with it or not.
Activity
vadimi commentedon Jan 23, 2026
hello @aldas , would it make sense to move Open Telemetry support into this repo from
open-telemetry/opentelemetry-go-contrib? I've opened a PR in there to support echo v5 (open-telemetry/opentelemetry-go-contrib#8457), but there is a question about the ownership and the actual location of the new package.What do you think? I'd be happy to open a PR against this repo
aldas commentedon Jan 23, 2026
@vadimi Hi, I'm probably a bit of a Luddite by today's standards, and I haven't used Open Telemetry before. That middleware part config.go+echo.go does not seem too complex. so we probably can support it. If you look at the commit history this repo is not that active, so I do not know if this suitable for otel tempo?
about
internal/semconvstuff in that PR - without me trying to understand that - is that also needed?vadimi commentedon Jan 23, 2026
internal/semconvis normally used for keeping otel attributes that will be used in spans/traces, it's autogenerated in open-telemetry repo, but it doesn't necessary need to be structured like that here.In terms of activity - I don't expect lots of changes and we can keep it as a sub-package to avoid pulling too many dependencies in the main
echo-contribmodule similar to how it's done in open-telemetry contrib repo or https://github.com/redis/go-redis/tree/master/extra/redisotelaldas commentedon Jan 25, 2026
Alright, @vadimi, please open PR in this repo.
Lets try sub-package out. We actually had this idea #52 couple of years ago.
vadimi commentedon Jan 26, 2026
Nice, opened this PR - #141.
One thing I'm not sure about is
github.com/labstack/echo-contrib/otelecho/v5path for the sub module to be able to introduce breaking changes into echo, but keep the previous versions still available. Please let me know if you're ok with it or not.aldas commentedon Feb 15, 2026
Echo OpenTelemetry middleware is located here https://github.com/labstack/echo-opentelemetry
aldas commentedon Feb 15, 2026
Please leave first impressions and feedback under this issue labstack/echo-otel#4