Adopt controller-runtime change PR3229: fake client cleared apiversion/kind

Signed-off-by: RainbowMango <[email protected]>

Author: RainbowMango

pkg/controllers/execution/execution_controller.go

@@ -358,6 +358,16 @@ func (c *Controller) updateWorkDispatchingConditionIfNeeded(ctx context.Context,
 		return err
 	}
 
+	if work.GetObjectKind().GroupVersionKind().Empty() {
+		// In unit tests, we need to verify how many events are emitted. However, when using the fake client,
+		// the work object may not have TypeMeta (APIVersion/Kind) set, which prevents building a proper
+		// object reference and thus no event is actually emitted. This workaround ensures the GVK is set
+		// so event emission, and thereby event counting in unit tests, works as expected.
+		// Since controller-runtime v0.22.0, the group version kind is not set when using fake client.
+		// See https://github.com/kubernetes-sigs/controller-runtime/pull/3229 for more details.
+		work.SetGroupVersionKind(workv1alpha1.SchemeGroupVersion.WithKind("Work"))
+	}
+
 	obj, err := helper.ToUnstructured(work)
 	if err != nil {
 		return err

pkg/controllers/namespace/namespace_sync_controller.go

@@ -82,6 +82,14 @@ func (c *Controller) Reconcile(ctx context.Context, req controllerruntime.Reques
 
 		return controllerruntime.Result{}, err
 	}
+	if namespace.GetObjectKind().GroupVersionKind().Empty() {
+		// In unit tests, we need to verify that a work object is created for the corresponding namespace.
+		// However, the fake client does not set TypeMeta information, which results in an incorrect generated work object name.
+		// This ensures correct GVK is set so work name generation in tests works as expected.
+		// Since controller-runtime v0.22.0, the group version kind is not set when using fake client.
+		// See https://github.com/kubernetes-sigs/controller-runtime/pull/3229 for more details.
+		namespace.SetGroupVersionKind(corev1.SchemeGroupVersion.WithKind("Namespace"))
+	}
 
 	if !namespace.DeletionTimestamp.IsZero() {
 		// Do nothing, just return as we have added owner reference to Work.