From a71078b73370cf7ad100e484e03a87285fcba6ef Mon Sep 17 00:00:00 2001 From: Travis Nielsen Date: Mon, 14 Oct 2024 17:04:54 -0600 Subject: [PATCH] core: skip counting exporter pods as a ceph pod The exporter pods were being counted as a ceph pod, so once an exporter pod was added to a node, it would never be removed since the controller thinks the exporter pod itself is a ceph pod for which the exporter must be present. Signed-off-by: Travis Nielsen --- pkg/operator/ceph/cluster/nodedaemon/add.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/operator/ceph/cluster/nodedaemon/add.go b/pkg/operator/ceph/cluster/nodedaemon/add.go index 9fae930d8d42..325114cb9598 100644 --- a/pkg/operator/ceph/cluster/nodedaemon/add.go +++ b/pkg/operator/ceph/cluster/nodedaemon/add.go @@ -177,7 +177,8 @@ func isCephPod(labels map[string]string, podName string) bool { // will be empty since the monitors don't exist yet isCanaryPod := strings.Contains(podName, "-canary-") isCrashCollectorPod := strings.Contains(podName, "-crashcollector-") - if ok && !isCanaryPod && !isCrashCollectorPod { + isExporterPod := strings.Contains(podName, "-exporter-") + if ok && !isCanaryPod && !isCrashCollectorPod && !isExporterPod { logger.Debugf("%q is a ceph pod!", podName) return true }