Skip to content

Commit

Permalink
Throttler flaky test: explicitly disabling throttler so as to ensure …
Browse files Browse the repository at this point in the history
…it does not re-enable (vitessio#16369)

Signed-off-by: Shlomi Noach <[email protected]>
  • Loading branch information
shlomi-noach authored Jul 11, 2024
1 parent ba5297d commit 75aee8a
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions go/vt/vttablet/tabletserver/throttle/throttler_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -771,6 +771,16 @@ func TestApplyThrottlerConfigAppCheckedMetrics(t *testing.T) {
assert.Len(t, checkResult.Metrics, 1)
})
})

t.Run("Disable", func(t *testing.T) {
throttlerConfig := &topodatapb.ThrottlerConfig{
Enabled: false,
MetricThresholds: map[string]float64{},
AppCheckedMetrics: map[string]*topodatapb.ThrottlerConfig_MetricNames{},
}
throttler.applyThrottlerConfig(ctx, throttlerConfig)
sleepTillThresholdApplies()
})
})
}

Expand Down

0 comments on commit 75aee8a

Please sign in to comment.