Skip to content

Reload stable object to prevent erase artifacts with old state (Ver 2)

Shinya Maeda requested to merge fix/sm/fix_pages_worker 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

Edited by Shinya Maeda

Merge request reports