Skip to content

Commit 2a28176

Browse files
committed
Fix highcpu intervals picking up the wrong e2etest intervals
1 parent 74c8e4a commit 2a28176

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/monitortests/testframework/highcputestanalyzer/monitortest.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,9 +86,9 @@ func findE2EIntervalsOverlappingHighCPU(intervals monitorapi.Intervals) []map[st
8686
interval.Message.Reason == monitorapi.IntervalReason("HighCPUUsage")
8787
})
8888

89-
// Filter for E2E test intervals
89+
// Filter for E2E test intervals, but the summary ones, not the started/finished variants
9090
e2eTestIntervals := intervals.Filter(func(interval monitorapi.Interval) bool {
91-
return interval.Source == monitorapi.SourceE2ETest
91+
return interval.Source == monitorapi.SourceE2ETest && interval.Display
9292
})
9393

9494
// Find E2E tests that overlap with alert intervals

0 commit comments

Comments
 (0)