Merge pull request #1794 from jedevc/fix-printer-start-race

docker · May 12, 2023 · 060ac84 · 060ac84
2 parents 31d1b77 + be65429
commit 060ac84
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/util/progress/printer.go b/util/progress/printer.go
@@ -120,9 +120,8 @@ func NewPrinter(ctx context.Context, w io.Writer, out console.File, mode string,
 			pw.logSourceMap = map[digest.Digest]interface{}{}
 			pw.logMu.Unlock()
 
-			close(pw.ready)
-
 			resumeLogs := logutil.Pause(logrus.StandardLogger())
+			close(pw.ready)
 			// not using shared context to not disrupt display but let is finish reporting errors
 			pw.warnings, pw.err = progressui.DisplaySolveStatus(ctx, c, w, pw.status, opt.displayOpts...)
 			resumeLogs()