cleanup

Signed-off-by: Aditi Ahuja <ahuja.aditi@gmail.com>

pkg/tracing/client/factory.go

@@ -34,12 +34,12 @@ import (
 type TracingProvider string
 
 const (
-	Stackdriver TracingProvider = "STACKDRIVER"
-	GoogleCloud TracingProvider = "GOOGLE_CLOUD"
-	Jaeger      TracingProvider = "JAEGER"
-	ElasticAPM  TracingProvider = "ELASTIC_APM"
-	Lightstep   TracingProvider = "LIGHTSTEP"
-	name                        = "thanos"
+	Stackdriver           TracingProvider = "STACKDRIVER"
+	GoogleCloud           TracingProvider = "GOOGLE_CLOUD"
+	Jaeger                TracingProvider = "JAEGER"
+	ElasticAPM            TracingProvider = "ELASTIC_APM"
+	Lightstep             TracingProvider = "LIGHTSTEP"
+	OpenTelemetryProtocol TracingProvider = "OTLP"
 )
 
 type TracingConfig struct {

pkg/tracing/jaeger/jaeger.go

@@ -150,7 +150,6 @@ func NewTracer(ctx context.Context, logger log.Logger, metrics *prometheus.Regis
 		return nil, nil, err
 	}
 
-	level.Info(logger).Log("msg", "getting tracing config", cfg)
 	cfg.Headers = &jaeger.HeadersConfig{
 		JaegerDebugHeader: strings.ToLower(tracing.ForceTracingBaggageKey),
 	}
@@ -165,12 +164,6 @@ func NewTracer(ctx context.Context, logger log.Logger, metrics *prometheus.Regis
 		return nil, nil, err
 	}
 
-	exp, err := otel_jaeger.New(otel_jaeger.WithCollectorEndpoint())
-	if err != nil {
-		return nil, nil, err
-	}
-	_ = exp
-
 	t := &Tracer{
 		jaegerTracer,
 	}

test/e2e/tracing_test.go

@@ -1,3 +1,5 @@
+// Copyright (c) The Thanos Authors.
+// Licensed under the Apache License 2.0.
 package e2e_test
 
 import (
@@ -13,7 +15,7 @@ import (
 	"gopkg.in/yaml.v2"
 )
 
-// test to check if the trace provider works as expected
+// Test to check if the trace provider works as expected.
 func TestJaegerTracing(t *testing.T) {
 	env, err := e2e.NewDockerEnvironment("e2e-tracing-test")
 	testutil.Ok(t, err)