remove todo

hashicorp · Aug 8, 2023 · a928007 · a928007
1 parent 714662c
commit a928007
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/internal/catalog/internal/controllers/failover/status.go b/internal/catalog/internal/controllers/failover/status.go
@@ -12,7 +12,8 @@ const (
 	StatusKey              = "consul.io/failover-policy"
 	StatusConditionHealthy = "healthy"
 
-	OKReason = "Ok"
+	OKReason  = "Ok"
+	OKMessage = "failover policy is valid"
 
 	MissingServiceReason  = "MissingService"
 	MissingServiceMessage = "service for failover policy does not exist"
@@ -32,10 +33,10 @@ const (
 
 var (
 	ConditionOK = &pbresource.Condition{
-		Type:   StatusConditionHealthy,
-		State:  pbresource.Condition_STATE_TRUE,
-		Reason: OKReason,
-		// TODO: needs message?
+		Type:    StatusConditionHealthy,
+		State:   pbresource.Condition_STATE_TRUE,
+		Reason:  OKReason,
+		Message: OKMessage,
 	}
 
 	ConditionMissingService = &pbresource.Condition{