Add more context to tailer-> ingester connect error. (#5394)

NOTE: When client is tailing, Querier **has** to create grpc connections to **all** the ingesters.

Sometimes when ingester is rolloing out and Querier unable to connect to ingester when
client is tailing, then error message reported by Querier is confusing.

This PR add some context to it.

Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com>
pull/5306/merge
Kaviraj Kanagaraj 3 years ago committed by GitHub
parent b3af2dbeef
commit d970e737ab
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 3
      pkg/querier/tail.go

@ -2,6 +2,7 @@ package querier
import ( import (
"context" "context"
"fmt"
"sync" "sync"
"time" "time"
@ -172,7 +173,7 @@ func (t *Tailer) checkIngesterConnections() error {
newConnections, err := t.tailDisconnectedIngesters(connectedIngestersAddr) newConnections, err := t.tailDisconnectedIngesters(connectedIngestersAddr)
if err != nil { if err != nil {
return err return fmt.Errorf("failed to connect with one or more ingester(s) during tailing: %w", err)
} }
if len(newConnections) != 0 { if len(newConnections) != 0 {

Loading…
Cancel
Save