Merge pull request #11494 from fabriziopandini/refine-v1beta2-nodehealthy-condition

🌱 Refine v1beta2 NodeHealthy condition

Author: k8s-ci-robot

internal/controllers/machine/machine_controller_status.go

@@ -26,6 +26,7 @@ import (
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
+	"k8s.io/apimachinery/pkg/util/sets"
 	"k8s.io/utils/ptr"
 	ctrl "sigs.k8s.io/controller-runtime"
 	"sigs.k8s.io/controller-runtime/pkg/client"
@@ -426,6 +427,8 @@ func summarizeNodeV1Beta2Conditions(_ context.Context, node *corev1.Node) (metav
 	semanticallyFalseStatus := 0
 	unknownStatus := 0
 
+	conditionCount := 0
+	conditionMessages := sets.Set[string]{}
 	messages := []string{}
 	for _, conditionType := range []corev1.NodeConditionType{corev1.NodeReady, corev1.NodeMemoryPressure, corev1.NodeDiskPressure, corev1.NodePIDPressure} {
 		var condition *corev1.NodeCondition
@@ -447,6 +450,8 @@ func summarizeNodeV1Beta2Conditions(_ context.Context, node *corev1.Node) (metav
 				if m == "" {
 					m = fmt.Sprintf("Condition is %s", condition.Status)
 				}
+				conditionCount++
+				conditionMessages.Insert(m)
 				messages = append(messages, fmt.Sprintf("* Node.%s: %s", condition.Type, m))
 				if condition.Status == corev1.ConditionUnknown {
 					unknownStatus++
@@ -461,6 +466,8 @@ func summarizeNodeV1Beta2Conditions(_ context.Context, node *corev1.Node) (metav
 				if m == "" {
 					m = fmt.Sprintf("Condition is %s", condition.Status)
 				}
+				conditionCount++
+				conditionMessages.Insert(m)
 				messages = append(messages, fmt.Sprintf("* Node.%s: %s", condition.Type, m))
 				if condition.Status == corev1.ConditionUnknown {
 					unknownStatus++
@@ -471,6 +478,9 @@ func summarizeNodeV1Beta2Conditions(_ context.Context, node *corev1.Node) (metav
 		}
 	}
 
+	if conditionCount > 1 && len(conditionMessages) == 1 {
+		messages = []string{fmt.Sprintf("* Node.AllConditions: %s", conditionMessages.UnsortedList()[0])}
+	}
 	message := strings.Join(messages, "\n")
 	if semanticallyFalseStatus > 0 {
 		return metav1.ConditionFalse, clusterv1.MachineNodeNotHealthyV1Beta2Reason, message

internal/controllers/machine/machine_controller_status_test.go

@@ -558,12 +558,9 @@ func TestSummarizeNodeV1Beta2Conditions(t *testing.T) {
 				{Type: corev1.NodeDiskPressure, Status: corev1.ConditionUnknown, Message: "Node is not reporting status"},
 				{Type: corev1.NodePIDPressure, Status: corev1.ConditionUnknown, Message: "Node is not reporting status"},
 			},
-			expectedStatus: metav1.ConditionUnknown,
-			expectedReason: clusterv1.MachineNodeHealthUnknownV1Beta2Reason,
-			expectedMessage: "* Node.Ready: Node is not reporting status\n" +
-				"* Node.MemoryPressure: Node is not reporting status\n" +
-				"* Node.DiskPressure: Node is not reporting status\n" +
-				"* Node.PIDPressure: Node is not reporting status",
+			expectedStatus:  metav1.ConditionUnknown,
+			expectedReason:  clusterv1.MachineNodeHealthUnknownV1Beta2Reason,
+			expectedMessage: "* Node.AllConditions: Node is not reporting status",
 		},
 		{
 			name: "multiple semantically failed condition",