Improve CC RefVersionsUpToDate condition message

Signed-off-by: Stefan Büringer buringerst@vmware.com

Author: sbueringer

internal/controllers/clusterclass/clusterclass_controller_status.go

@@ -50,9 +50,12 @@ func setRefVersionsUpToDateCondition(_ context.Context, clusterClass *clusterv1.
 	}
 
 	if len(outdatedRefs) > 0 {
-		var msg []string
+		var msg = []string{
+			"The following templateRefs are not using the latest apiVersion:",
+		}
 		for _, outdatedRef := range outdatedRefs {
-			msg = append(msg, fmt.Sprintf("* templateRef %q should be %q", refString(outdatedRef.Outdated), refString(outdatedRef.UpToDate)))
+			msg = append(msg, fmt.Sprintf("* %s %s: used: %s, available: %s", outdatedRef.Outdated.Kind, outdatedRef.Outdated.Name,
+				outdatedRef.Outdated.GroupVersionKind().Version, outdatedRef.UpToDate.GroupVersionKind().Version))
 		}
 		v1beta1conditions.Set(clusterClass,
 			v1beta1conditions.FalseCondition(

internal/controllers/clusterclass/clusterclass_controller_status_test.go

@@ -81,10 +81,9 @@ func TestSetRefVersionsUpToDateCondition(t *testing.T) {
 				Type:   clusterv1.ClusterClassRefVersionsUpToDateCondition,
 				Status: metav1.ConditionFalse,
 				Reason: clusterv1.ClusterClassRefVersionsNotUpToDateReason,
-				Message: "* templateRef \"controlplane.cluster.x-k8s.io/v1beta2, Kind=KubeadmControlPlaneTemplate default/test-kcp\" should be " +
-					"\"controlplane.cluster.x-k8s.io/v99, Kind=KubeadmControlPlaneTemplate default/test-kcp\"\n" +
-					"* templateRef \"infrastructure.cluster.x-k8s.io/v1beta2, Kind=DockerMachineTemplate default/test-dmt\" should be " +
-					"\"infrastructure.cluster.x-k8s.io/v99, Kind=DockerMachineTemplate default/test-dmt\"",
+				Message: "The following templateRefs are not using the latest apiVersion:\n" +
+					"* KubeadmControlPlaneTemplate test-kcp: used: v1beta2, available: v99\n" +
+					"* DockerMachineTemplate test-dmt: used: v1beta2, available: v99",
 			},
 		},
 		{