From 9dcc90ecd7a71d2926ac75463f9a6d17f56647df Mon Sep 17 00:00:00 2001 From: Luca Comellini Date: Thu, 6 Jun 2024 07:19:49 -0700 Subject: [PATCH] Add wrapcheck (#124) --- .golangci.yml | 4 +++- pkg/telemetry/span_provider.go | 7 ++++++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/.golangci.yml b/.golangci.yml index dd7ece9..7829405 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -75,6 +75,7 @@ linters: - noctx - nolintlint - paralleltest + - perfsprint - prealloc - predeclared - reassign @@ -91,7 +92,8 @@ linters: - unused - usestdlibvars - wastedassign - - perfsprint + - whitespace + - wrapcheck disable-all: true issues: max-issues-per-linter: 0 diff --git a/pkg/telemetry/span_provider.go b/pkg/telemetry/span_provider.go index 4a44b96..aa169e1 100644 --- a/pkg/telemetry/span_provider.go +++ b/pkg/telemetry/span_provider.go @@ -2,6 +2,7 @@ package telemetry import ( "context" + "fmt" "go.opentelemetry.io/otel/exporters/otlp/otlptrace" "go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc" @@ -23,5 +24,9 @@ func CreateOTLPSpanProvider(options ...otlptracegrpc.Option) SpanProvider { // newOTLPExporter creates a new gRPC OTLP exporter. func newOTLPExporter(ctx context.Context, options ...otlptracegrpc.Option) (*otlptrace.Exporter, error) { traceClient := otlptracegrpc.NewClient(options...) - return otlptrace.New(ctx, traceClient) + exp, err := otlptrace.New(ctx, traceClient) + if err != nil { + return nil, fmt.Errorf("failed to create OTLP exporter: %w", err) + } + return exp, nil }