Remove otel workaround test

dmcgowan · dmcgowan · commit 8875d6a01a30 · 2025-01-13T23:09:47.000-08:00
Signed-off-by: Derek McGowan &lt;derek@mcg.dev&gt;
diff --git a/cmd/dockerd/daemon_test.go b/cmd/dockerd/daemon_test.go
@@ -1,14 +1,12 @@
 package main
 
 import (
-	"runtime"
 	"testing"
 
 	"github.com/containerd/log"
 	"github.com/docker/docker/daemon/config"
 	"github.com/google/go-cmp/cmp/cmpopts"
 	"github.com/spf13/pflag"
-	"go.opentelemetry.io/otel"
 	"gotest.tools/v3/assert"
 	is "gotest.tools/v3/assert/cmp"
 	"gotest.tools/v3/fs"
@@ -281,29 +279,3 @@ func TestCDISpecDirs(t *testing.T) {
 		})
 	}
 }
-
-// TestOtelMeterLeak tests for a memory leak in the OTEL meter implementation.
-// Once the fixed OTEL is vendored, this test will fail - the workaround
-// and this test should be removed then.
-func TestOtelMeterLeak(t *testing.T) {
-	meter := otel.Meter("foo")
-
-	var before runtime.MemStats
-	runtime.ReadMemStats(&before)
-
-	const counters = 10 * 1000 * 1000
-	for i := 0; i < counters; i++ {
-		_, _ = meter.Int64Counter("bar")
-	}
-
-	var after runtime.MemStats
-	runtime.ReadMemStats(&after)
-
-	allocs := after.Mallocs - before.Mallocs
-	t.Log("Allocations:", allocs)
-
-	if allocs < 10 {
-		// TODO: Remove Workaround OTEL memory leak in cmd/dockerd/daemon.go
-		t.Fatal("Allocations count decreased. OTEL leak workaround is no longer needed!")
-	}
-}
