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)
Edited by Steve Xuereb