blender/flamenco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Worker: Blender command, don't put current time in log lines

Browse Source 
The time has to be put in at a different abstraction level.
This commit is contained in:
Sybren A. Stüvel 2022-02-22 15:36:20 +01:00
parent adf7113b34
commit a650aa4cbd
1 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
internal/worker/command_blender.go
View File

@ -93,14 +93,7 @@ func (ce *CommandExecutor) cmdBlenderRender(ctx context.Context, logger zerolog.
} }
logger.Debug().Msg(line) logger.Debug().Msg(line)
logChunker.Append(ctx, fmt.Sprintf("pid=%d > %s", blenderPID, line))
timestamp := ce.timeService.Now().Format(timeFormat)
// %35s because trailing zeroes in the nanoseconds aren't output by the
// formatted timestamp, and thus it has a variable length. Using a fixed
// width in this Sprintf() call ensures the rest of the line aligns visually
// with the preceeding ones.
logLine := fmt.Sprintf("%35s: pid=%d > %s", timestamp, blenderPID, line)
logChunker.Append(ctx, logLine)
} }
logChunker.Flush(ctx) logChunker.Flush(ctx)