Skip to content
Snippets Groups Projects
Select Git revision
  • master default protected
  • 12-9-stable
  • 12-7-stable
  • 12-6-stable
  • 12-8-stable
  • github/fork/Kloppi313/patch-1
  • 12-5-stable
  • 12-4-stable
  • github/fork/ramalokesh8477/master
  • 12-1-stable
  • 12-2-stable
  • 12-0-stable
  • 12-3-stable
  • 42-42-stable
  • github/fork/hussamgit398/patch-2
  • 12-3-auto-deploy-20190911
  • 12-3-auto-deploy-20190916
  • 12-3-auto-deploy-20190908
  • 12-3-auto-deploy-20190901
  • 12-3-auto-deploy-20190901-32664
  • v12.10.0.pre
  • v12.9.0
  • v12.9.0-rc42
  • v12.8.7
  • v12.8.6
  • v12.8.5
  • v12.8.4
  • v12.8.3
  • v12.6.8
  • v12.7.7
  • v12.8.2
  • v12.8.1
  • v12.9.0.pre
  • v12.8.0
  • v12.8.0-rc42
  • v12.5.10
  • v12.7.6
  • v12.6.7
  • v12.7.5
  • v12.5.9
40 results

sh-conditional-system-hook-push.yml

  • Stan Hu's avatar
    377e3670
    Eliminate unnecessary and duplicate system hook fires · 377e3670
    Stan Hu authored
    Previously `SystemHookPushWorker` would always be called after a push event,
    and this would queue a Sidekiq job regardless of whether any system hooks
    needed that event. Moreover, another call inside `Project#execute_hooks` would
    also fire system hooks if they existed.
    
    This change both removes the duplicate system hook calls. For installations
    without system hooks for push events, this change also can save significant
    amount of RAM used by Redis.
    
    Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/50549
    377e3670
    History
    Eliminate unnecessary and duplicate system hook fires
    Stan Hu authored
    Previously `SystemHookPushWorker` would always be called after a push event,
    and this would queue a Sidekiq job regardless of whether any system hooks
    needed that event. Moreover, another call inside `Project#execute_hooks` would
    also fire system hooks if they existed.
    
    This change both removes the duplicate system hook calls. For installations
    without system hooks for push events, this change also can save significant
    amount of RAM used by Redis.
    
    Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/50549