Fix error message for max tail connections. (#2161)

* Fix error message for max tail connections.

Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>

* Fixes a test.

Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>

pkg/querier/querier.go

@@ -559,10 +559,10 @@ func (q *Querier) checkTailRequestLimit(ctx context.Context) error {
 			maxCnt = r
 		}
 	}
-
-	if maxCnt >= uint32(q.limits.MaxConcurrentTailRequests(userID)) {
+	l := uint32(q.limits.MaxConcurrentTailRequests(userID))
+	if maxCnt >= l {
 		return httpgrpc.Errorf(http.StatusBadRequest,
-			"max concurrent tail requests limit exceeded, count > limit (%d > %d)", maxCnt+1, 1)
+			"max concurrent tail requests limit exceeded, count > limit (%d > %d)", maxCnt+1, l)
 	}
 
 	return nil

pkg/querier/querier_test.go

@@ -558,7 +558,7 @@ func TestQuerier_concurrentTailLimits(t *testing.T) {
 		"ring containing one active ingester and max active tailers": {
 			ringIngesters: []ring.IngesterDesc{mockIngesterDesc("1.1.1.1", ring.ACTIVE)},
 			expectedError: httpgrpc.Errorf(http.StatusBadRequest,
-				"max concurrent tail requests limit exceeded, count > limit (%d > %d)", 6, 1),
+				"max concurrent tail requests limit exceeded, count > limit (%d > %d)", 6, 5),
 			tailersCount: 5,
 		},
 	}