Skip to content

Remove debug logger interfaces in executors/docker/internal

Lyubomir Raykov requested to merge docker-networks-manager-logger into master

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

Merge request reports