Merge pull request #9133 from killianmuldoon/pr-fix-clusterclass-enqueue

k8s-ci-robot · web-flow · commit 2061cabe5677 · 2023-08-07T08:46:32.000-07:00
🐛 Fix ClusterClass enqueue for ExtensionConfig
diff --git a/internal/controllers/clusterclass/clusterclass_controller.go b/internal/controllers/clusterclass/clusterclass_controller.go
@@ -409,7 +409,7 @@ func (r *Reconciler) extensionConfigToClusterClass(ctx context.Context, o client
 		}
 		for _, patch := range clusterClass.Spec.Patches {
 			if patch.External != nil && patch.External.DiscoverVariablesExtension != nil {
-				res = append(res, ctrl.Request{NamespacedName: client.ObjectKey{Name: ext.Name}})
+				res = append(res, ctrl.Request{NamespacedName: client.ObjectKey{Namespace: clusterClass.Namespace, Name: clusterClass.Name}})
 				break
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -409,7 +409,7 @@ func (r *Reconciler) extensionConfigToClusterClass(ctx context.Context, o client`
`409`	`409`	`}`
`410`	`410`	`for _, patch := range clusterClass.Spec.Patches {`
`411`	`411`	`if patch.External != nil && patch.External.DiscoverVariablesExtension != nil {`
`412`		`- res = append(res, ctrl.Request{NamespacedName: client.ObjectKey{Name: ext.Name}})`
	`412`	`+ res = append(res, ctrl.Request{NamespacedName: client.ObjectKey{Namespace: clusterClass.Namespace, Name: clusterClass.Name}})`
`413`	`413`	`break`
`414`	`414`	`}`
`415`	`415`	`}`