diff options
author | Stan Hu <stanhu@gmail.com> | 2018-09-19 15:37:02 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2018-09-19 16:36:29 +0300 |
commit | 311beef242ae4fc3dd666542ca01938fb9d0e15d (patch) | |
tree | 4dcaf0d71384b5b0c856c74145d846a606be4f76 /app | |
parent | a1912ccc894386b112faba2932f1dd98c03aea0e (diff) |
Move registry destroy out of project transaction
Diffstat (limited to 'app')
-rw-r--r-- | app/models/container_repository.rb | 2 | ||||
-rw-r--r-- | app/services/projects/destroy_service.rb | 19 |
2 files changed, 15 insertions, 6 deletions
diff --git a/app/models/container_repository.rb b/app/models/container_repository.rb index 93ca3a5daf4..2c08a8e1acf 100644 --- a/app/models/container_repository.rb +++ b/app/models/container_repository.rb @@ -8,8 +8,6 @@ class ContainerRepository < ActiveRecord::Base delegate :client, to: :registry - before_destroy :delete_tags! - # rubocop: disable CodeReuse/ServiceClass def registry @registry ||= begin diff --git a/app/services/projects/destroy_service.rb b/app/services/projects/destroy_service.rb index 5090ebf8f51..210571b6b4e 100644 --- a/app/services/projects/destroy_service.rb +++ b/app/services/projects/destroy_service.rb @@ -133,11 +133,11 @@ module Projects end def attempt_destroy_transaction(project) - Project.transaction do - unless remove_legacy_registry_tags - raise_error('Failed to remove some tags in project container registry. Please try again or contact administrator.') - end + unless remove_registry_tags + raise_error('Failed to remove some tags in project container registry. Please try again or contact administrator.') + end + Project.transaction do log_destroy_event trash_repositories! @@ -156,6 +156,17 @@ module Projects log_info("Attempting to destroy #{project.full_path} (#{project.id})") end + def remove_registry_tags + return false unless remove_legacy_registry_tags + + project.container_repositories.find_each do |container_repository| + service = Projects::ContainerRepository::DestroyService.new(project, current_user) + service.execute(container_repository) + end + + true + end + ## # This method makes sure that we correctly remove registry tags # for legacy image repository (when repository path equals project path). |