Skip to content

Remove hooks backup test from backup.rake specs

James Fargher requested to merge gitaly_disk_access_backup_rake into master

What does this MR do and why?

#375254 (closed)

This removes a test that was accessing a gitaly managed filesystem to check that a backup had worked correctly. This will not be possible when we start using gitaly cluster by default as it will be impossible to determine where the file is located.

In gitaly#4438 (comment 1112606409) the gitaly team effectively decided that we are okay removing coverage from gitlab-rails as long as it is also covered by gitaly itself.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by James Fargher

Merge request reports

Loading