feedback

hashicorp · Nov 30, 2022 · 4372a52 · 4372a52
1 parent 65c70e8
commit 4372a52
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/agent/grpc-external/services/peerstream/subscription_manager.go b/agent/grpc-external/services/peerstream/subscription_manager.go
@@ -672,8 +672,11 @@ var statusScores = map[string]int{
 	api.HealthPassing:  4,
 }
 
-func isStatusBetter(curr, next string) bool {
-	return statusScores[next] < statusScores[curr]
+func getMostImportantStatus(a, b string) string {
+	if statusScores[a] < statusScores[b] {
+		return a
+	}
+	return b
 }
 
 func flattenChecks(
@@ -696,9 +699,7 @@ func flattenChecks(
 				healthStatus = api.HealthMaint
 				break // always wins
 			}
-			if isStatusBetter(healthStatus, chk.Status) {
-				healthStatus = chk.Status
-			}
+			healthStatus = getMostImportantStatus(healthStatus, chk.Status)
 		}
 	}