Skip to content

Commit 93d2ed0

Browse files
authored
clean up ingester per labelset metrics (#6439)
Signed-off-by: Ben Ye <[email protected]>
1 parent 81321dc commit 93d2ed0

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

pkg/ingester/ingester_test.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -498,6 +498,13 @@ func TestIngesterPerLabelsetLimitExceeded(t *testing.T) {
498498
cortex_ingester_usage_per_labelset{labelset="{label2=\"value2\"}",limit="max_series",user="1"} 2
499499
cortex_ingester_usage_per_labelset{labelset="{}",limit="max_series",user="1"} 10
500500
`), "cortex_ingester_usage_per_labelset", "cortex_ingester_limits_per_labelset"))
501+
502+
// Force set tenant to be deleted.
503+
ing.getTSDB(userID).deletionMarkFound.Store(true)
504+
require.Equal(t, tsdbTenantMarkedForDeletion, ing.closeAndDeleteUserTSDBIfIdle(userID))
505+
// LabelSet metrics cleaned up.
506+
require.NoError(t, testutil.GatherAndCompare(registry, bytes.NewBufferString(``), "cortex_ingester_usage_per_labelset", "cortex_ingester_limits_per_labelset"))
507+
501508
services.StopAndAwaitTerminated(context.Background(), ing) //nolint:errcheck
502509

503510
}

pkg/ingester/metrics.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,8 @@ func (m *ingesterMetrics) deletePerUserMetrics(userID string) {
278278
m.memMetadataCreatedTotal.DeleteLabelValues(userID)
279279
m.memMetadataRemovedTotal.DeleteLabelValues(userID)
280280
m.activeSeriesPerUser.DeleteLabelValues(userID)
281+
m.usagePerLabelSet.DeletePartialMatch(prometheus.Labels{"user": userID})
282+
m.limitsPerLabelSet.DeletePartialMatch(prometheus.Labels{"user": userID})
281283

282284
if m.memSeriesCreatedTotal != nil {
283285
m.memSeriesCreatedTotal.DeleteLabelValues(userID)

0 commit comments

Comments
 (0)