[chore][exporter/pulsar] Enable goleak (#33135)

crobert-1 · web-flow · commit eb0669039345 · 2024-05-21T12:28:05.000-07:00
This is a test only change, adding `goleak` to help ensure no goroutines are being leaked. This passed in local testing without any changes. #30438 `goleak` is passing
diff --git a/exporter/pulsarexporter/generated_package_test.go b/exporter/pulsarexporter/generated_package_test.go
diff --git a/exporter/pulsarexporter/go.mod b/exporter/pulsarexporter/go.mod
@@ -20,6 +20,7 @@ require (
 	go.opentelemetry.io/collector/semconv v0.101.0
 	go.opentelemetry.io/otel/metric v1.26.0
 	go.opentelemetry.io/otel/trace v1.26.0
+	go.uber.org/goleak v1.3.0
 	go.uber.org/multierr v1.11.0
 	go.uber.org/zap v1.27.0
 )
diff --git a/exporter/pulsarexporter/metadata.yaml b/exporter/pulsarexporter/metadata.yaml
@@ -13,4 +13,7 @@ status:
 tests:
   skip_lifecycle: true
   goleak:
-    skip: true
+    ignore:
+      any:
+        # Regarding the godbus/dbus ignore: see https://github.com/99designs/keyring/issues/103
+        - "github.com/godbus/dbus.(*Conn).inWorker"

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@ require (`
`20`	`20`	`go.opentelemetry.io/collector/semconv v0.101.0`
`21`	`21`	`go.opentelemetry.io/otel/metric v1.26.0`
`22`	`22`	`go.opentelemetry.io/otel/trace v1.26.0`
	`23`	`+ go.uber.org/goleak v1.3.0`
`23`	`24`	`go.uber.org/multierr v1.11.0`
`24`	`25`	`go.uber.org/zap v1.27.0`
`25`	`26`	`)`