diff options
| author | Stan Hu <stanhu@gmail.com> | 2018-08-21 13:32:50 -0700 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2018-08-21 13:39:02 -0700 |
| commit | 377e3670a9aa2ca1419232d75b060cfccc58c7ef (patch) | |
| tree | 24a4a01c29bc3dcb3caaf214114dbfc90e9ade9e /changelogs/unreleased/sh-conditional-system-hook-push.yml | |
| parent | 382b6dabd96e8966386729d82854b899ad4ce894 (diff) | |
| download | gitlab-ce-377e3670a9aa2ca1419232d75b060cfccc58c7ef.tar.gz | |
Eliminate unnecessary and duplicate system hook fires
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
Diffstat (limited to 'changelogs/unreleased/sh-conditional-system-hook-push.yml')
| -rw-r--r-- | changelogs/unreleased/sh-conditional-system-hook-push.yml | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/changelogs/unreleased/sh-conditional-system-hook-push.yml b/changelogs/unreleased/sh-conditional-system-hook-push.yml new file mode 100644 index 00000000000..3a1a1b3d36c --- /dev/null +++ b/changelogs/unreleased/sh-conditional-system-hook-push.yml @@ -0,0 +1,5 @@ +--- +title: Eliminate unnecessary and duplicate system hook fires +merge_request: 21337 +author: +type: performance |