Skip to content

Read archived build logs with database reads consistency [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Grzegorz Bizon requested to merge fix/gb/consistent-archived-trace-reads into master

What does this MR do?

This merge request makes it possible to avoid a race condition that manifests itself in an exception that bubbles up to Sentry https://sentry.gitlab.net/gitlab/gitlabcom/issues/2441626

We want to avoid the race between archiving a build log and a user accessing it. It may happen that a log has been archived when a request to review it happens, and therefore build trace chunks are being removed exactly when a user tries to access them.

Edited by 🤖 GitLab Bot 🤖

Merge request reports