From 86e210a6f9b818ef6e170f405bb19680ce63a2b5 Mon Sep 17 00:00:00 2001
From: Anna Kapuscinska <anna@isovalent.com>
Date: Tue, 2 Apr 2024 18:38:53 +0100
Subject: [PATCH] metrics: Remove pod from the queue after deleting metrics

This is done to prevent a memory leak in environments with a high pod churn or
very long-running Tetragon agents.

Credits to Tam Mach for fixing a similar issue in cilium/cilium#31714.

Signed-off-by: Anna Kapuscinska <anna@isovalent.com>
---
 pkg/metrics/metricwithpod.go | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pkg/metrics/metricwithpod.go b/pkg/metrics/metricwithpod.go
index 4e036831727..92e7fb29032 100644
--- a/pkg/metrics/metricwithpod.go
+++ b/pkg/metrics/metricwithpod.go
@@ -128,5 +128,6 @@ func StartPodDeleteHandler() {
 			return
 		}
 		DeleteMetricsForPod(pod.(*corev1.Pod))
+		queue.Done(pod)
 	}
 }