Commit 7c88cd5d authored by Igor Drozdov's avatar Igor Drozdov

Merge branch 'delete-pages-configuration-worker' into 'master'

Remove PagesUpdateConfigurationWorker logic

See merge request gitlab-org/gitlab!78449
parents 94c78fd6 457d53c1
......@@ -836,7 +836,6 @@ Gitlab/NamespacedClass:
- app/workers/pages_domain_verification_cron_worker.rb
- app/workers/pages_domain_verification_worker.rb
- app/workers/pages_transfer_worker.rb
- app/workers/pages_update_configuration_worker.rb
- app/workers/pages_worker.rb
- app/workers/partition_creation_worker.rb
- app/workers/pipeline_hooks_worker.rb
......
......@@ -2623,15 +2623,6 @@
:weight: 1
:idempotent:
:tags: []
- :name: pages_update_configuration
:worker_name: PagesUpdateConfigurationWorker
:feature_category: :pages
:has_external_dependencies:
:urgency: :low
:resource_boundary: :unknown
:weight: 1
:idempotent: true
:tags: []
- :name: phabricator_import_import_tasks
:worker_name: Gitlab::PhabricatorImport::ImportTasksWorker
:feature_category: :importers
......
# frozen_string_literal: true
# TODO: remove this in 14.7 https://gitlab.com/gitlab-org/gitlab/-/issues/348582
class PagesUpdateConfigurationWorker
include ApplicationWorker
data_consistency :always
sidekiq_options retry: 1
idempotent!
feature_category :pages
def perform(_project_id)
# Do nothing
end
end
......@@ -311,8 +311,6 @@
- 1
- - pages_transfer
- 1
- - pages_update_configuration
- 1
- - personal_access_tokens
- 1
- - phabricator_import_import_tasks
......
......@@ -370,7 +370,6 @@ RSpec.describe 'Every Sidekiq worker' do
'PagesDomainSslRenewalWorker' => 3,
'PagesDomainVerificationWorker' => 3,
'PagesTransferWorker' => 3,
'PagesUpdateConfigurationWorker' => 1,
'PagesWorker' => 3,
'PersonalAccessTokens::Groups::PolicyWorker' => 3,
'PersonalAccessTokens::Instance::PolicyWorker' => 3,
......
# frozen_string_literal: true
require "spec_helper"
RSpec.describe PagesUpdateConfigurationWorker do
let_it_be(:project) { create(:project) }
describe "#perform" do
it "does not break" do
expect { subject.perform(-1) }.not_to raise_error
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