From ce58f78d60ee759bbe05446f9f02a19be6094cf6 Mon Sep 17 00:00:00 2001 From: Eric Date: Fri, 5 Jun 2026 13:50:04 -0400 Subject: [PATCH 1/2] Added a if-statement to compare images Signed-off-by: Eric --- pkg/controller/build/reconciler.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/pkg/controller/build/reconciler.go b/pkg/controller/build/reconciler.go index 79d91de71e..09e510666e 100644 --- a/pkg/controller/build/reconciler.go +++ b/pkg/controller/build/reconciler.go @@ -1390,6 +1390,12 @@ func (b *buildReconciler) reconcilePoolChange(ctx context.Context, mcp *mcfgv1.M // In both cases, we need to create/continue a build missingAnnotation := firstOptIn == "" + // No action needed if the rendered config has not changed. + if oldRendered == newRendered { + klog.V(4).Infof("pool %q: Configuration unchanged (%s), no action needed", mcp.Name, oldRendered) + return nil + } + // This is our trigger point // Create/continue a build if: // 1. Config changed AND needs image rebuild, OR From 23696fbfd1783e6b01dcbb4f6f42eedbe2388270 Mon Sep 17 00:00:00 2001 From: Eric Date: Fri, 5 Jun 2026 14:44:41 -0400 Subject: [PATCH 2/2] Added check for missingAnnotation --- pkg/controller/build/reconciler.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/controller/build/reconciler.go b/pkg/controller/build/reconciler.go index 09e510666e..6e961026c9 100644 --- a/pkg/controller/build/reconciler.go +++ b/pkg/controller/build/reconciler.go @@ -1391,7 +1391,7 @@ func (b *buildReconciler) reconcilePoolChange(ctx context.Context, mcp *mcfgv1.M missingAnnotation := firstOptIn == "" // No action needed if the rendered config has not changed. - if oldRendered == newRendered { + if oldRendered == newRendered && !missingAnnotation { klog.V(4).Infof("pool %q: Configuration unchanged (%s), no action needed", mcp.Name, oldRendered) return nil }