Commit bfc0965d authored by Olivier Bornet's avatar Olivier Bornet

Prevent runner to be stealth with long logs.

If the log is longer than maxTraceOutputSize, the runner will be paused
and waiting forever. This ia because the reader stop reading the pipe,
and the pipes has a size limit.

I have already proposed a patch in the past for this. The patch was
adapted, but unfortunately, an important "break" was not removed in the
adaptation.

This patch remove the unwanted break, allowing the for loop to continue
to run and flushing the pipe buffer.
parent 45ed89c6
Pipeline #94755 failed with stage
......@@ -62,7 +62,6 @@ func (e *AbstractExecutor) ReadTrace(pipe *io.PipeReader) {
)
e.Build.WriteString(output)
traceStopped = true
break
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment