Update addon available condition message (#239)

zhujian7 · web-flow · commit 13fbb6259464 · 2024-02-05T01:37:30.000Z
Signed-off-by: zhujian &lt;jiazhu@redhat.com&gt;
diff --git a/pkg/addonmanager/controllers/agentdeploy/healthcheck_sync.go b/pkg/addonmanager/controllers/agentdeploy/healthcheck_sync.go
@@ -192,7 +192,7 @@ func (s *healthCheckSyncer) probeAddonStatusByWorks(
 		Type:    addonapiv1alpha1.ManagedClusterAddOnConditionAvailable,
 		Status:  metav1.ConditionTrue,
 		Reason:  addonapiv1alpha1.AddonAvailableReasonProbeAvailable,
-		Message: "Addon is available",
+		Message: fmt.Sprintf("%s add-on is available.", addon.Name),
 	})
 	return nil
 }
diff --git a/pkg/addonmanager/controllers/agentdeploy/healthcheck_sync_test.go b/pkg/addonmanager/controllers/agentdeploy/healthcheck_sync_test.go
@@ -290,7 +290,7 @@ func TestHealthCheckReconcile(t *testing.T) {
 				Type:    addonapiv1alpha1.ManagedClusterAddOnConditionAvailable,
 				Status:  metav1.ConditionTrue,
 				Reason:  addonapiv1alpha1.AddonAvailableReasonProbeAvailable,
-				Message: "Addon is available",
+				Message: "test add-on is available.",
 			},
 		},
 
@@ -527,7 +527,7 @@ func TestHealthCheckReconcile(t *testing.T) {
 				Type:    addonapiv1alpha1.ManagedClusterAddOnConditionAvailable,
 				Status:  metav1.ConditionTrue,
 				Reason:  addonapiv1alpha1.AddonAvailableReasonProbeAvailable,
-				Message: "Addon is available",
+				Message: "test add-on is available.",
 			},
 		},
 	}

Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,7 @@ func (s *healthCheckSyncer) probeAddonStatusByWorks(`
`192`	`192`	`Type: addonapiv1alpha1.ManagedClusterAddOnConditionAvailable,`
`193`	`193`	`Status: metav1.ConditionTrue,`
`194`	`194`	`Reason: addonapiv1alpha1.AddonAvailableReasonProbeAvailable,`
`195`		`- Message: "Addon is available",`
	`195`	`+ Message: fmt.Sprintf("%s add-on is available.", addon.Name),`
`196`	`196`	`})`
`197`	`197`	`return nil`
`198`	`198`	`}`