Skip to content

Commit ac05dc7

Browse files
authored
Fixed regression introduced in #4240 (#4251)
Signed-off-by: Marco Pracucci <marco@pracucci.com>
1 parent 3fb0d4d commit ac05dc7

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

integration/querier_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -962,9 +962,10 @@ func TestQueryLimitsWithBlocksStorageRunningInMicroServices(t *testing.T) {
962962
res, err = c.Push(series4)
963963
require.NoError(t, err)
964964
require.Equal(t, 200, res.StatusCode)
965+
965966
_, err = c.QueryRange("{__name__=~\"series_.+\"}", series1Timestamp, series4Timestamp.Add(1*time.Hour), blockRangePeriod)
966967
require.Error(t, err)
967-
assert.Contains(t, err.Error(), "500")
968+
assert.Contains(t, err.Error(), "max number of series limit")
968969
}
969970

970971
func TestHashCollisionHandling(t *testing.T) {

pkg/distributor/query.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ func (d *Distributor) queryIngesterStream(ctx context.Context, userID string, re
245245

246246
for _, series := range resp.Timeseries {
247247
if limitErr := queryLimiter.AddSeries(series.Labels); limitErr != nil {
248-
return nil, limitErr
248+
return nil, validation.LimitError(limitErr.Error())
249249
}
250250
}
251251

0 commit comments

Comments
 (0)