Fix metrics data race in the Engine test run finalization

This should close #1887
grafana · Mar 8, 2021 · 2036bae · 2036bae
1 parent f1413e6
commit 2036bae
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/core/engine.go b/core/engine.go
@@ -328,6 +328,15 @@ func (e *Engine) processMetrics(globalCtx context.Context, processMetricsAfterRu
 		case <-ticker.C:
 			processSamples()
 		case <-processMetricsAfterRun:
+		getCachedMetrics:
+			for {
+				select {
+				case sc := <-e.Samples:
+					sampleContainers = append(sampleContainers, sc)
+				default:
+					break getCachedMetrics
+				}
+			}
 			e.logger.Debug("Processing metrics and thresholds after the test run has ended...")
 			processSamples()
 			if !e.runtimeOptions.NoThresholds.Bool {