[chore] Remove direct dependency on jaeger-lib (#4721)

## Which problem is this PR solving? - Part of #3381 ## Description of the changes - Remove internal unused package `jlibadapter` - Remove unnecessary dependency on jaeger-lib in the tests ## How was this change tested? - Unit tests Signed-off-by: Yuri Shkuro <[email protected]>
jaegertracing · Sep 4, 2023 · a49a164 · a49a164
1 parent ecffad7
commit a49a164
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 114 deletions.
diff --git a/go.mod b/go.mod
@@ -42,7 +42,6 @@ require (
 	github.com/spf13/viper v1.16.0
 	github.com/stretchr/testify v1.8.4
 	github.com/uber/jaeger-client-go v2.30.0+incompatible
-	github.com/uber/jaeger-lib v2.4.1+incompatible
 	github.com/xdg-go/scram v1.1.2
 	go.opentelemetry.io/collector/component v0.84.0
 	go.opentelemetry.io/collector/config/configgrpc v0.84.0
@@ -151,6 +150,7 @@ require (
 	github.com/spf13/jwalterweatherman v1.1.0 // indirect
 	github.com/stretchr/objx v0.5.0 // indirect
 	github.com/subosito/gotenv v1.4.2 // indirect
+	github.com/uber/jaeger-lib v2.4.1+incompatible // indirect
 	github.com/xdg-go/pbkdf2 v1.0.0 // indirect
 	github.com/xdg-go/stringprep v1.0.4 // indirect
 	go.mongodb.org/mongo-driver v1.11.6 // indirect

diff --git a/internal/metrics/expvar/metrics_test.go b/internal/metrics/expvar/metrics_test.go
@@ -20,9 +20,9 @@ import (
 
 	"github.com/go-kit/kit/metrics/generic"
 	"github.com/stretchr/testify/assert"
-	"github.com/uber/jaeger-lib/metrics"
 
 	"github.com/jaegertracing/jaeger/internal/metrics/expvar"
+	"github.com/jaegertracing/jaeger/pkg/metrics"
 )
 
 func TestCounter(t *testing.T) {

diff --git a/internal/metrics/jlibadapter/adapter.go b/internal/metrics/jlibadapter/adapter.go
diff --git a/internal/metrics/jlibadapter/adapter_test.go b/internal/metrics/jlibadapter/adapter_test.go