Skip to content

Remove specific logger for network manager

The following discussion from !1569 (merged) should be addressed:

  • @tmaczukin started a discussion:

    @steveazz Maybe we should use a FieldLogger interface for such usage?

Proposal

Remove the added debugLogger interfaces and replace them with logrus.FieldLogger. Also instead of passing in the BuildLogger, t use e.Build.Log(), which should have the same fields as the BuildLogger but doesn't go to JobTrace.

Example merge request in !1920 (closed)