Merge branch '52560-fix-duplicate-tag-system-hooks' into 'master'

Only execute system hooks once when pushing tags

Closes #52560

See merge request gitlab-org/gitlab-ce!26888

1 files changed, 0 insertions, 12 deletions

diff --git a/app/services/git/tag_push_service.rb b/app/services/git/tag_push_service.rb
index 318dfd4f886..9ce0fbdb206 100644
--- a/app/services/git/tag_push_service.rb
+++ b/app/services/git/tag_push_service.rb
@@ -13,7 +13,6 @@ module Git
       EventCreateService.new.push(project, current_user, push_data)
       Ci::CreatePipelineService.new(project, current_user, push_data).execute(:push, pipeline_options)
 
-      SystemHooksService.new.execute_hooks(build_system_push_data, :tag_push_hooks)
       project.execute_hooks(push_data.dup, :tag_push_hooks)
       project.execute_services(push_data.dup, :tag_push_hooks)
 
@@ -50,17 +49,6 @@ module Git
         push_options: params[:push_options] || [])
     end
 
-    def build_system_push_data
-      Gitlab::DataBuilder::Push.build(
-        project,
-        current_user,
-        params[:oldrev],
-        params[:newrev],
-        params[:ref],
-        [],
-        '')
-    end
-
     def pipeline_options
       {} # to be overridden in EE
     end