diff options
author | Sean McGivern <sean@gitlab.com> | 2017-07-17 18:38:02 +0300 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2017-07-17 18:38:02 +0300 |
commit | ae10151b4234372b4b1ce0f46ecc29b782cec1e0 (patch) | |
tree | 16e4914c666f3c9b1035b3da9c3969ef01b8a6bf /app/workers | |
parent | 05329d4a364a5c55f2de9546871de1909b6be3f5 (diff) |
Skip dead jobs queue for web hooks and project services
These jobs may legitimately fail because the endpoint just isn't there any
more. We don't want them cluttering up the Sidekiq morgue in that case - it's
currently full on GitLab.com.
Diffstat (limited to 'app/workers')
-rw-r--r-- | app/workers/project_service_worker.rb | 2 | ||||
-rw-r--r-- | app/workers/web_hook_worker.rb | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/app/workers/project_service_worker.rb b/app/workers/project_service_worker.rb index fdfdeab7b41..4883d848c53 100644 --- a/app/workers/project_service_worker.rb +++ b/app/workers/project_service_worker.rb @@ -2,6 +2,8 @@ class ProjectServiceWorker include Sidekiq::Worker include DedicatedSidekiqQueue + sidekiq_options dead: false + def perform(hook_id, data) data = data.with_indifferent_access Service.find(hook_id).execute(data) diff --git a/app/workers/web_hook_worker.rb b/app/workers/web_hook_worker.rb index ad5ddf02a12..713c0228040 100644 --- a/app/workers/web_hook_worker.rb +++ b/app/workers/web_hook_worker.rb @@ -2,7 +2,7 @@ class WebHookWorker include Sidekiq::Worker include DedicatedSidekiqQueue - sidekiq_options retry: 4 + sidekiq_options retry: 4, dead: false def perform(hook_id, data, hook_name) hook = WebHook.find(hook_id) |