Merge branch 'zj-repo-cleanup-mandatory' into 'master'

Make repository cleanup mandatory

Closes #49550

See merge request gitlab-org/gitlab-ce!20808
This commit is contained in:
Douwe Maan 2018-07-24 12:45:28 +00:00
commit adc327d3fa
1 changed files with 2 additions and 4 deletions

View File

@ -353,8 +353,6 @@ module Gitlab
# offset: 5,
# after: Time.new(2016, 4, 21, 14, 32, 10)
# )
#
# Gitaly migration: https://gitlab.com/gitlab-org/gitaly/issues/446
def log(options)
default_options = {
limit: 10,
@ -1029,8 +1027,8 @@ module Gitlab
end
def clean_stale_repository_files
gitaly_migrate(:repository_cleanup, status: Gitlab::GitalyClient::MigrationStatus::OPT_OUT) do |is_enabled|
gitaly_repository_client.cleanup if is_enabled && exists?
wrapped_gitaly_errors do
gitaly_repository_client.cleanup if exists?
end
rescue Gitlab::Git::CommandError => e # Don't fail if we can't cleanup
Rails.logger.error("Unable to clean repository on storage #{storage} with relative path #{relative_path}: #{e.message}")