fix: health hpa when only second conditions is False

pkg/health/health_hpa.go

@@ -3,6 +3,7 @@ package health
 import (
 	"encoding/json"
 	"fmt"
+	"strings"
 
 	autoscalingv1 "k8s.io/api/autoscaling/v1"
 	autoscalingv2 "k8s.io/api/autoscaling/v2"
@@ -132,6 +133,7 @@ func getAutoScalingV1HPAHealth(hpa *autoscalingv1.HorizontalPodAutoscaler) (*Hea
 }
 
 func checkConditions(conditions []hpaCondition, progressingStatus *HealthStatus) (*HealthStatus, error) {
+	healthyMessages := []string{}
 	for _, condition := range conditions {
 		if isDegraded(&condition) {
 			return &HealthStatus{
@@ -141,13 +143,14 @@ func checkConditions(conditions []hpaCondition, progressingStatus *HealthStatus)
 		}
 
 		if isHealthy(&condition) {
-			return &HealthStatus{
-				Status:  HealthStatusHealthy,
-				Message: condition.Message,
-			}, nil
+			healthyMessages = append(healthyMessages, condition.Message)
 		}
 	}
 
+	if len(healthyMessages) > 0 {
+		return &HealthStatus{Status: HealthStatusHealthy, Message: strings.Join(healthyMessages, ",")}, nil
+	}
+
 	return progressingStatus, nil
 }
 

pkg/health/health_test.go

@@ -81,12 +81,13 @@ func TestJob(t *testing.T) {
 func TestHPA(t *testing.T) {
 	assertAppHealth(t, "./testdata/hpa-v2-healthy.yaml", HealthStatusHealthy)
 	assertAppHealth(t, "./testdata/hpa-v2-degraded.yaml", HealthStatusDegraded)
+	assertAppHealth(t, "./testdata/hpa-v2-degraded-partially.yaml", HealthStatusDegraded)
 	assertAppHealth(t, "./testdata/hpa-v2-progressing.yaml", HealthStatusProgressing)
 	assertAppHealth(t, "./testdata/hpa-v2beta2-healthy.yaml", HealthStatusHealthy)
 	assertAppHealth(t, "./testdata/hpa-v2beta1-healthy-disabled.yaml", HealthStatusHealthy)
 	assertAppHealth(t, "./testdata/hpa-v2beta1-healthy.yaml", HealthStatusHealthy)
 	assertAppHealth(t, "./testdata/hpa-v1-degraded.yaml", HealthStatusDegraded)
-	assertAppHealth(t, "./testdata/hpa-v1-healthy.yaml", HealthStatusHealthy)
+	assertAppHealth(t, "./testdata/hpa-v1-degraded-partially.yaml", HealthStatusDegraded)
 	assertAppHealth(t, "./testdata/hpa-v1-healthy-toofew.yaml", HealthStatusHealthy)
 	assertAppHealth(t, "./testdata/hpa-v1-progressing.yaml", HealthStatusProgressing)
 	assertAppHealth(t, "./testdata/hpa-v1-progressing-with-no-annotations.yaml", HealthStatusProgressing)

pkg/health/testdata/hpa-v2-degraded-partially.yaml

@@ -0,0 +1,40 @@
+apiVersion: autoscaling/v2
+kind: HorizontalPodAutoscaler
+metadata:
+  creationTimestamp: "2022-01-17T14:22:27Z"
+  name: sample
+  uid: 0e6d855e-83ed-4ed5-b80a-461a750f14db
+spec:
+  maxReplicas: 2
+  minReplicas: 1
+  scaleTargetRef:
+    apiVersion: apps/v1
+    kind: Deployment
+    name: argocd-server
+  targetCPUUtilizationPercentage: 80
+status:
+  conditions:
+  - lastTransitionTime: '2024-09-05T20:20:41Z'
+    message: the HPA controller was able to get the target's current scale
+    reason: SucceededGetScale
+    status: 'True'
+    type: AbleToScale
+  - lastTransitionTime: '2024-09-05T20:20:56Z'
+    message: |
+      the HPA was unable to compute the replica count: failed to get memory
+      utilization: unable to get metrics for resource memory: unable to fetch
+      metrics from resource metrics API: the server could not find the
+      requested resource (get pods.metrics.k8s.io)
+    reason: FailedGetResourceMetric
+    status: 'False'
+    type: ScalingActive
+  type: ScalingLimited
+  currentMetrics:
+  - resource:
+      current:
+        averageUtilization: 6
+        averageValue: 12m
+      name: cpu
+    type: Resource
+  currentReplicas: 1
+  desiredReplicas: 1