Skip to content

Commit 1d7be73

Browse files
authored
Fix flaky TestPerLabelsetDiscardedSeriesTracker (#7036)
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
1 parent 1bc480f commit 1d7be73

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pkg/util/discardedseries/perlabelset_tracker.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,9 @@ func (t *DiscardedSeriesPerLabelsetTracker) Track(user string, series uint64, ma
8181

8282
func (t *DiscardedSeriesPerLabelsetTracker) UpdateMetrics() {
8383
usersToDelete := make([]string, 0)
84-
labelsetsToDelete := make([]uint64, 0)
8584
t.RLock()
8685
for user, labelsetCounter := range t.userLabelsetMap {
86+
labelsetsToDelete := make([]uint64, 0)
8787
labelsetCounter.RLock()
8888
if len(labelsetCounter.labelsetSeriesMap) == 0 {
8989
usersToDelete = append(usersToDelete, user)

pkg/util/discardedseries/perlabelset_tracker_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,5 +114,5 @@ func TestPerLabelsetDiscardedSeriesTracker(t *testing.T) {
114114

115115
func comparePerLabelsetSeriesVendedCount(t *testing.T, gaugeVec *prometheus.GaugeVec, user string, labelsetLimitId string, val int) {
116116
gauge, _ := gaugeVec.GetMetricWithLabelValues("per_labelset_series_limit", user, labelsetLimitId)
117-
require.Equal(t, testutil.ToFloat64(gauge), float64(val))
117+
require.Equal(t, float64(val), testutil.ToFloat64(gauge))
118118
}

0 commit comments

Comments
 (0)