Skip to content

Update some logrus fields used in Runner logs

Tomasz Maczukin requested to merge update-log-fields into master

What does this MR do?

Updates few fields logged with structured logging

Why was this MR needed?

As described in !1361 (merged), some software like Elasticsearch doesn't like when the same field have different types. This MR adds two more fixes to resolve such problem.

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?

Merge request reports