diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2018-06-26 19:24:42 +0300 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2018-07-02 13:48:57 +0300 |
commit | 75316348c50d03d9aed760e9b603275995e4e3e3 (patch) | |
tree | 3eeaebf24fceb6d378063f592682ace0b35533d6 /spec/workers | |
parent | f25cdea64d69a97f31719622f9dead3de1ea0e11 (diff) |
Prune web hook logs older than 90 days
This adds a recurring Sidekiq job that removes up to 50 000 old web hook
logs per hour, if they are older than 90 days. This will prevent the
web_hook_logs table from growing indefinitely.
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/46120
Diffstat (limited to 'spec/workers')
-rw-r--r-- | spec/workers/prune_web_hook_logs_worker_spec.rb | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/spec/workers/prune_web_hook_logs_worker_spec.rb b/spec/workers/prune_web_hook_logs_worker_spec.rb new file mode 100644 index 00000000000..d7d64a1f641 --- /dev/null +++ b/spec/workers/prune_web_hook_logs_worker_spec.rb @@ -0,0 +1,22 @@ +require 'spec_helper' + +describe PruneWebHookLogsWorker do + describe '#perform' do + before do + hook = create(:project_hook) + + 5.times do + create(:web_hook_log, web_hook: hook, created_at: 5.months.ago) + end + + create(:web_hook_log, web_hook: hook, response_status: '404') + end + + it 'removes all web hook logs older than one month' do + described_class.new.perform + + expect(WebHookLog.count).to eq(1) + expect(WebHookLog.first.response_status).to eq('404') + end + end +end |