Handle the situation when one of the nodes returns an error

Signed-off-by: Mikolaj Pawlikowski <[email protected]>
matmerr · Mar 12, 2021 · 634e04e · 634e04e
1 parent 13ae09d
commit 634e04e
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/pkg/goldpinger/client.go b/pkg/goldpinger/client.go
@@ -89,6 +89,12 @@ func CheckCluster(ctx context.Context) *models.ClusterHealthResults {
 		response.OK = false
 	} else {
 		for _, resp := range checkAll.Responses {
+			// on error, there might be no response from the node
+			if resp.Response == nil {
+				response.OK = false
+				break
+			}
+			// if we get a response, let's check we get the expected nodes
 			observedNodes := []string{}
 			for peer := range resp.Response.PodResults {
 				observedNodes = append(observedNodes, peer)