Remove debug logger interfaces in executors/docker/internal
What does this MR do?
Removes the added debugLogger
interfaces and replaces them with logrus.FieldLogger
. Also instead of passing in the BuildLogger
, they now use e.Build.Log()
, which should have the same fields as the BuildLogger
but doesn't go to JobTrace
.
Also in a few places changes string interpolated data into fields for better structured logging.
Why was this MR needed?
To go in the direction of #6609.
Are there points in the code the reviewer needs to double check?
Does this MR meet the acceptance criteria?
-
Documentation created/updated -
Added tests for this feature/bug -
In case of conflicts with master
- branch was rebased
What are the relevant issue numbers?
Fixes #11925
Edited by Lyubomir Raykov