mudhorn/prometheus
1
0
Fork 0
mirror of https://github.com/prometheus/prometheus.git synced 2025-02-21 03:16:00 -08:00
Code Issues Actions 4 Packages Projects Releases Wiki Activity

Merge pull request #13380 from machine424/fixes

Browse source 
fix(discovery tests): allow requireUpdate util to timeout in discovery/file…
This commit is contained in:
Bryan Boreham 2024-01-18 14:11:01 +00:00 committed by GitHub
parent bfbb13cf36 afc05129f1
commit b9952ff571
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
discovery/file/file_test.go
View file

@ -193,9 +193,10 @@ func (t *testRunner) targets() []*targetgroup.Group {
func (t *testRunner) requireUpdate(ref time.Time, expected []*targetgroup.Group) { func (t *testRunner) requireUpdate(ref time.Time, expected []*targetgroup.Group) {
t.Helper() t.Helper()
timeout := time.After(defaultWait)
for { for {
select { select {
case <-time.After(defaultWait): case <-timeout:
t.Fatalf("Expected update but got none") t.Fatalf("Expected update but got none")
return return
case <-time.After(defaultWait / 10): case <-time.After(defaultWait / 10):