From 20923257407d705fee0d5b3238f441402214f8cd Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Fri, 8 Jun 2018 13:33:51 -0700 Subject: Add more logging before a project is destroyed This adds a simple log message but also allows EE to insert database events within the transaction (https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/6059/diffs). --- app/services/projects/destroy_service.rb | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/app/services/projects/destroy_service.rb b/app/services/projects/destroy_service.rb index de0125ed0dd..02769e72229 100644 --- a/app/services/projects/destroy_service.rb +++ b/app/services/projects/destroy_service.rb @@ -135,6 +135,7 @@ module Projects raise_error('Failed to remove some tags in project container registry. Please try again or contact administrator.') end + log_destroy_event trash_repositories! # Rails attempts to load all related records into memory before @@ -148,6 +149,10 @@ module Projects end end + def log_destroy_event + log_info("Attempting to destroy #{project.full_path} (#{project.id})") + end + ## # This method makes sure that we correctly remove registry tags # for legacy image repository (when repository path equals project path). -- cgit v1.2.1