Return original enpoints when all are evicted

Signed-off-by: JmPotato <ghzpotato@gmail.com>

pkg/utils/etcdutil/health_checker.go

@@ -329,6 +329,14 @@ func (checker *healthChecker) filterEps(eps []string) []string {
 		}
 		pickedEps = append(pickedEps, ep)
 	}
+	// If the pickedEps is empty, it means all endpoints are evicted,
+	// to gain a better availability, just use the original picked endpoints.
+	if len(pickedEps) == 0 {
+		log.Warn("all etcd endpoints are evicted, use the picked endpoints directly",
+			zap.Strings("endpoints", eps),
+			zap.String("source", checker.source))
+		return eps
+	}
 	return pickedEps
 }
 

pkg/utils/etcdutil/health_checker_test.go

@@ -201,7 +201,7 @@ func TestPickEps(t *testing.T) {
 				},
 			},
 			map[string]int{"A": 0, "B": 1, "C": 1, "D": 0},
-			[]string{},
+			[]string{"B", "C"},
 		},
 	}
 	check(re, testCases)
@@ -318,7 +318,7 @@ func TestLatencyPick(t *testing.T) {
 				},
 			},
 			map[string]int{"A": 1, "B": 1, "C": 0},
-			[]string{},
+			[]string{"A", "B"},
 		},
 	}
 	check(re, testCases)