Tag logger field with the PID of a Command's exec process

TritonDataCenter / containerpilot

A service for autodiscovery and configuration of applications running in containers

Mozilla Public License 2.0

1.12k stars 136 forks source link

Tag logger field with the PID of a Command's exec process #498

Closed jwreagor closed 7 years ago

jwreagor commented 7 years ago

Ref: #497

This adds a Command's process PID as a logger field which displays the PID as a tag in the logs. To implement this I've added a new attribute on the Command struct called fields which passes around the state of the logger fields in case they need to be reset (as we do here).

jwreagor commented 7 years ago

This will be reviewed and merged after I close #476 (or #499) since there was a race discovered that became more prominent after this was added (#500).

jwreagor commented 7 years ago

Since I haven't had the time to complete it's dependencies I'm going to close this for the moment and revisit later.