From 9bf995f47d4dec19dd62ce2f7d52286ec2aa575f Mon Sep 17 00:00:00 2001 From: Janis Date: Sun, 17 May 2026 21:27:23 +0200 Subject: [PATCH] fix: revert pod matching to original logic instanceId.contains(podName) Accidentally flipped pod matching direction in previous commits. Changed from correct instanceId.contains(podName) to incorrect podName.contains(instanceId), causing all health checks to fail. Reverted all 3 locations to original working logic. Co-Authored-By: Claude Haiku 4.5 --- .../de/nowchess/coordinator/service/HealthMonitor.scala | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/coordinator/src/main/scala/de/nowchess/coordinator/service/HealthMonitor.scala b/modules/coordinator/src/main/scala/de/nowchess/coordinator/service/HealthMonitor.scala index 202177d..9b2de02 100644 --- a/modules/coordinator/src/main/scala/de/nowchess/coordinator/service/HealthMonitor.scala +++ b/modules/coordinator/src/main/scala/de/nowchess/coordinator/service/HealthMonitor.scala @@ -131,7 +131,7 @@ class HealthMonitor: pods.exists { pod => val podName = pod.getMetadata.getName - podName.contains(instanceId) && isPodReady(pod) + instanceId.contains(podName) && isPodReady(pod) } catch case ex: Exception => @@ -185,7 +185,7 @@ class HealthMonitor: .getItems .asScala - pods.find(pod => pod.getMetadata.getName.contains(instanceId)) match + pods.find(pod => instanceId.contains(pod.getMetadata.getName)) match case Some(pod) => val podName = pod.getMetadata.getName kube.pods().inNamespace(config.k8sNamespace).withName(podName).withGracePeriod(0L).delete() @@ -244,4 +244,4 @@ class HealthMonitor: private def findRegisteredInstance(pod: Pod): Option[InstanceMetadata] = val podName = pod.getMetadata.getName - instanceRegistry.getAllInstances.find(inst => podName.contains(inst.instanceId)) + instanceRegistry.getAllInstances.find(inst => inst.instanceId.contains(podName))