Skip to content

Commit

Permalink
Merge pull request kubernetes#788 from mm4tt/fix_etcd_prometheus
Browse files Browse the repository at this point in the history
Fix logic for checking if at least one etcd target is available
  • Loading branch information
k8s-ci-robot committed Sep 12, 2019
2 parents 826d9fe + 37ac016 commit cb7a940
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion clusterloader2/pkg/prometheus/prometheus.go
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@ func (pc *PrometheusController) isPrometheusReady() (bool, error) {
}
return CheckTargetsReady( // 1 out of 2 etcd targets should be ready.
pc.framework.GetClientSets().GetClient(),
func(t Target) bool { return !isEtcdEndpoint(t.Labels["endpoint"]) },
func(t Target) bool { return isEtcdEndpoint(t.Labels["endpoint"]) },
2, // expected targets: etcd-2379 and etcd-2382
1) // one of them should be healthy
}
Expand Down

0 comments on commit cb7a940

Please sign in to comment.