Skip to content

[BREAKING CHANGE] Refactor LoggerWrapper for better outputs to LogStash

Ecarg requested to merge grace/T364413/update-logging-to-standard into main
  • This removes the helper functions (that do not do anything unique) so that we can just use one 'standard' to log (i.e. logger.log()), rather than having two ways to do the same things and mixing them up everywhere.
  • This removes misleading three params of log(), when actually service-runner only considers the level and an object type.
  • This adds two beneficial info to our logger data: a timestamp and stack trace, regardless of level.
  • A console.log() has also been added so the same log info can be output to our docker logs when we run things locally. This way, we can mimic what would be seen in Logstash during local test/debugging.

Bug: T369001
Bug: T369213
Bug: T364413

Edited by Ecarg

Merge request reports