Skip to content

Do not log warning if trace update interval header value is empty

Pedro Pombeiro requested to merge remove-trace into master

What does this MR do?

It prevents logging a normal situation as a warning.

Why was this MR needed?

I saw this warning in the log, but this is a benign situation:

WARNING: Failed to parse "X-GitLab-Trace-Update-Interval" header  error=strconv.Atoi: parsing "": invalid syntax header-value= job=737 runner=pYJpZ6jY

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