Add wrapcheck (#124)

nginxinc · Jun 6, 2024 · 9dcc90e · 9dcc90e
1 parent b5af296
commit 9dcc90e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git 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
@@ -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
 }