Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove dependencies on Jaeger from opentelemetry-collector-contrib #6408

Open
yurishkuro opened this issue Dec 25, 2024 · 0 comments
Open

Remove dependencies on Jaeger from opentelemetry-collector-contrib #6408

yurishkuro opened this issue Dec 25, 2024 · 0 comments
Labels
area/otel help wanted Features that maintainers are willing to accept but do not have cycles to implement

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Dec 25, 2024

Today we have circular dependencies between Jaeger and OTEL contrib repos. We want to move towards one way Jaeger->OTEL/contrib dependency model. This requires refactoring of how Jaeger internal code is organized and moving some of the code out to OTEL/contrib.

This is a relatively large undertaking that needs to happen piecemeal.

Here are all the current OTEL/contrib -> Jaeger dependencies (Dec-2024):

$ rg -g '*.go' -g '!*_test.go' -N 'github.com\/jaegertracing' | grep -E '^\w' | sed 's/^.*"\(.*\)"$/\1/g' | grep '^github.com' | sort -u

github.com/jaegertracing/jaeger/cmd/agent/app/configmanager
github.com/jaegertracing/jaeger/cmd/agent/app/configmanager/grpc
github.com/jaegertracing/jaeger/cmd/agent/app/httpserver
github.com/jaegertracing/jaeger/cmd/agent/app/processors
github.com/jaegertracing/jaeger/cmd/agent/app/servers
github.com/jaegertracing/jaeger/cmd/agent/app/servers/thriftudp
github.com/jaegertracing/jaeger/cmd/collector/app/sampling
github.com/jaegertracing/jaeger/cmd/collector/app/sampling/samplingstrategy
github.com/jaegertracing/jaeger/model
github.com/jaegertracing/jaeger/model/converter/thrift/zipkin
github.com/jaegertracing/jaeger/pkg/cache
github.com/jaegertracing/jaeger/pkg/metrics
github.com/jaegertracing/jaeger/plugin/sampling/strategyprovider/static
github.com/jaegertracing/jaeger/plugin/storage/es/spanstore/dbmodel
github.com/jaegertracing/jaeger/proto-gen/api_v2
github.com/jaegertracing/jaeger/thrift-gen/agent
github.com/jaegertracing/jaeger/thrift-gen/baggage
github.com/jaegertracing/jaeger/thrift-gen/jaeger
github.com/jaegertracing/jaeger/thrift-gen/zipkincore
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/otel help wanted Features that maintainers are willing to accept but do not have cycles to implement
Projects
None yet
Development

No branches or pull requests

1 participant