Commit a3966076 authored by Matthias Käppler's avatar Matthias Käppler

Merge branch 'lm-update-pipeline-notification-worker-to-use-replica' into 'master'

Add data consistency for PipelineNotificationWorker

See merge request gitlab-org/gitlab!66459
parents b0efae90 4302ac2d
...@@ -3,13 +3,12 @@ ...@@ -3,13 +3,12 @@
class PipelineNotificationWorker # rubocop:disable Scalability/IdempotentWorker class PipelineNotificationWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker include ApplicationWorker
data_consistency :always
sidekiq_options retry: 3 sidekiq_options retry: 3
include PipelineQueue include PipelineQueue
urgency :high urgency :high
worker_resource_boundary :cpu worker_resource_boundary :cpu
data_consistency :delayed, feature_flag: :load_balancing_for_pipeline_notification_worker
def perform(pipeline_id, args = {}) def perform(pipeline_id, args = {})
case args case args
......
---
name: load_balancing_for_pipeline_notification_worker
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/66459
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/336517
milestone: '14.2'
type: development
group: group::pipeline authoring
default_enabled: false
...@@ -20,5 +20,10 @@ RSpec.describe PipelineNotificationWorker, :mailer do ...@@ -20,5 +20,10 @@ RSpec.describe PipelineNotificationWorker, :mailer do
subject.perform(non_existing_record_id) subject.perform(non_existing_record_id)
end end
it_behaves_like 'worker with data consistency',
described_class,
feature_flag: :load_balancing_for_pipeline_notification_worker,
data_consistency: :delayed
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment