Skip to content

Fix broken `use_file` method

Shinya Maeda requested to merge fix/sm/hard-reset-for-use-file into master

What does this MR do?

This fixes

  • ObjectStorage#use_file method has a bug that it doesn't erase cache file after used it.
  • Ci::Build#artifacts_metadata_entry is wrong as it generates metadata entry with use_file, but the entry is actually accessed out of the use_file block.

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?

  • Tests added for this feature/bug
  • Review
    • Has been reviewed by Backend
  • End-to-end tests pass (package-qa manual pipeline job)

What are the relevant issue numbers?

closes https://gitlab.com/gitlab-org/gitlab-ee/issues/4827

Edited by Shinya Maeda

Merge request reports