From 40d3afd14267c05eb0b0d44597423fbeb2759098 Mon Sep 17 00:00:00 2001 From: Ilya Date: Fri, 26 Nov 2021 17:16:57 +0000 Subject: [PATCH] Push result to target result channel without ctx.Done() check to avoid race condition. Signed-off-by: Ilya --- pkg/runner/test_runner.go | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/pkg/runner/test_runner.go b/pkg/runner/test_runner.go index 2aa28c56..7b3dbeea 100644 --- a/pkg/runner/test_runner.go +++ b/pkg/runner/test_runner.go @@ -588,12 +588,9 @@ func (tr *TestRunner) reportTargetResult(ss *stepState, tgt *target.Target, res if err != nil { return err } - select { - case resCh <- res: - break - case <-ss.ctx.Done(): - break - } + + // As it has buffer size 1 and we process a single result for each target at a time + resCh <- res return nil }