Skip to content

Reload stable object to prevent erase artifacts with old state

Shinya Maeda requested to merge fix/sm/fix_update_pages_service-ce into master

What does this MR do?

This fixes, when UpdatePagesService tried to erase artifacts, it raises an exception as Errno::ENOENT: No such file or directory, because it tries to erase it with old state.

For example, even if BackgroundMoveWorker updated file_store to 2 of ci_job_artifacts asynchronously, still UpdatePagesService reffers it as file_store is nil.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/43641

Merge request reports