add a phantomjs execution status to log if errors happens, e.g. OOM killer kills it (#9644)

fixes #9643

add a phantomjs execution status to log if errors happens, e.g. OOM killer kills it (#9644)
fixes #9643
bddebf4c · xjewer · Torkel Ödegaard · 89a07002 · bddebf4c
Commit bddebf4c authored Oct 25, 2017 by xjewer Committed by Torkel Ödegaard Oct 25, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletions

pkg/components/renderer/renderer.go
+3 -1

No files found.
--- a/pkg/components/renderer/renderer.go
+++ b/pkg/components/renderer/renderer.go
@@ -121,7 +121,9 @@ func RenderToPng(params *RenderOpts) (string, error) {

 	done := make(chan error)
 	go func() {
-		cmd.Wait()
+		if err := cmd.Wait(); err != nil {
+			rendererLog.Error("failed to render an image", "error", err)
+		}
 		close(done)
 	}()