Page MenuHomePhabricator

Logging: logs should include PID
Closed, ResolvedPublic

Description

Trying to interpret the Parsoid logging output is difficult because multiple workers' logs are multiplexed together. It would be great if the worker PID could be added to the start of the line, to make it possible to demultiplex the log file.


Version: unspecified
Severity: normal

Details

Reference
bz52358

Related Objects

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:09 AM
bzimport added a project: Parsoid.
bzimport set Reference to bz52358.

While doing this we should also replace all direct calls to console.log / console.warn / console.error with utility logging methods. We already have env.errCB and env is available widely, so we could probably add env.log and env.logError methods. Those could then be redefined depending on the context.

See also bug 49762 for other potential logging tweaks.

Now that we have output sent to Logstash via bunyan, this is done.