From 6bc70a2c263613f838bab1d7e9abb252544010c4 Mon Sep 17 00:00:00 2001 From: rut31337 Date: Sat, 9 May 2026 13:52:49 -0500 Subject: [PATCH] fix: use self.resource_pool_name in ResourcePoolScaling.manage() fallback The fallback path in manage() when ResourcePool.get() returns None references an undefined variable `name` instead of `self.resource_pool_name`. This causes a NameError that crashes the handler on every retry. The bug is masked when ResourcePool.get() succeeds from cache (line 103), but when the cache misses, the fallback on line 105 always fails: NameError: name 'name' is not defined. Did you mean: 'self.name'? This blocks all ResourcePoolScaling operations, preventing shared cluster pools from scaling up. --- operator/resourcepoolscaling.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/operator/resourcepoolscaling.py b/operator/resourcepoolscaling.py index 8c1b706..0c7646c 100644 --- a/operator/resourcepoolscaling.py +++ b/operator/resourcepoolscaling.py @@ -102,7 +102,7 @@ async def manage(self, logger: kopf.ObjectLogger) -> None: from resourcepool import ResourcePool resource_pool = await ResourcePool.get(self.resource_pool_name) if not resource_pool: - resource_pool = await ResourcePool.fetch(name=name, namespace=Poolboy.namespace) + resource_pool = await ResourcePool.fetch(name=self.resource_pool_name, namespace=Poolboy.namespace) if 'ownerReferences' not in self.metadata: await self.json_patch([{ "op": "add",