Skip to content

Add database timings to Sidekiq JSON logs

Sean McGivern requested to merge add-db-timings-to-sidekiq-logs into master

What does this MR do?

Sidekiq JSON logs have total duration, queuing time, Gitaly time, and CPU time. They don't (before this change) have database time.

We provide two fields: db_duration and db_duration_s. That's because the units between the different duration fields are currently confusing, so providing an explicit unit moves us closer to that goal, while keeping the raw figure in the un-suffixed fields.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Merge request reports