Commit bfb16f42 authored by Michael Kozono's avatar Michael Kozono

Merge branch 'ag-remove-ff-verification_state_backfill_worker' into 'master'

Remove FF `verification_state_backfill_worker`

See merge request gitlab-org/gitlab!72762
parents 6f6aedb5 9726601c
...@@ -51,7 +51,7 @@ module Geo ...@@ -51,7 +51,7 @@ module Geo
::Geo::ReverificationBatchWorker.perform_with_capacity(replicable_name) ::Geo::ReverificationBatchWorker.perform_with_capacity(replicable_name)
if Feature.enabled?(:verification_state_backfill_worker, default_enabled: :yaml) && verification_query_class.separate_verification_state_table? if verification_query_class.separate_verification_state_table?
::Geo::VerificationStateBackfillWorker.perform_async(replicable_name) ::Geo::VerificationStateBackfillWorker.perform_async(replicable_name)
end end
end end
......
---
name: verification_state_backfill_worker
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/69301
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/340648
milestone: '14.4'
type: development
group: group::geo
default_enabled: false
...@@ -144,18 +144,12 @@ RSpec.shared_examples 'a verifiable replicator' do ...@@ -144,18 +144,12 @@ RSpec.shared_examples 'a verifiable replicator' do
described_class.trigger_background_verification described_class.trigger_background_verification
end end
context 'when verification_state_backfill_worker feature flag is enabled' do
before do
stub_feature_flags(verification_state_backfill_worker: true)
expect(described_class.model).to receive(:separate_verification_state_table?).and_return(true)
end
it 'enqueues VerificationStateBackfillWorker' do it 'enqueues VerificationStateBackfillWorker' do
expect(described_class.model).to receive(:separate_verification_state_table?).and_return(true)
expect(::Geo::VerificationStateBackfillWorker).to receive(:perform_async).with(described_class.replicable_name) expect(::Geo::VerificationStateBackfillWorker).to receive(:perform_async).with(described_class.replicable_name)
described_class.trigger_background_verification described_class.trigger_background_verification
end end
end
context 'for a Geo secondary' do context 'for a Geo secondary' do
it 'does not enqueue ReverificationBatchWorker' do it 'does not enqueue ReverificationBatchWorker' do
......
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