Merge branch '340641-remove-pagesremoveworker' into 'master'

Remove PagesRemoveWorker

See merge request gitlab-org/gitlab!71073

.rubocop_manual_todo.yml

@@ -1769,7 +1769,6 @@ Gitlab/NamespacedClass:
     - 'app/workers/pages_domain_ssl_renewal_worker.rb'
     - 'app/workers/pages_domain_verification_cron_worker.rb'
     - 'app/workers/pages_domain_verification_worker.rb'
-    - 'app/workers/pages_remove_worker.rb'
     - 'app/workers/pages_transfer_worker.rb'
     - 'app/workers/pages_update_configuration_worker.rb'
     - 'app/workers/pages_worker.rb'

app/workers/all_queues.yml

@@ -2465,15 +2465,6 @@
   :weight: 1
   :idempotent:
   :tags: []
-- :name: pages_remove
-  :worker_name: PagesRemoveWorker
-  :feature_category: :pages
-  :has_external_dependencies:
-  :urgency: :low
-  :resource_boundary: :unknown
-  :weight: 1
-  :idempotent:
-  :tags: []
 - :name: pages_transfer
   :worker_name: PagesTransferWorker
   :feature_category: :pages

app/workers/pages_remove_worker.rb

-# frozen_string_literal: true
-
-# TODO: remove this worker https://gitlab.com/gitlab-org/gitlab/-/issues/340641
-class PagesRemoveWorker # rubocop:disable Scalability/IdempotentWorker
-  include ApplicationWorker
-
-  data_consistency :always
-
-  sidekiq_options retry: 3
-  feature_category :pages
-  loggable_arguments 0
-
-  def perform(project_id)
-    # no-op
-  end
-end

config/sidekiq_queues.yml

@@ -277,8 +277,6 @@
   - 1
 - - pages_domain_verification
   - 1
-- - pages_remove
-  - 1
 - - pages_transfer
   - 1
 - - pages_update_configuration

spec/workers/every_sidekiq_worker_spec.rb

@@ -368,7 +368,6 @@ RSpec.describe 'Every Sidekiq worker' do
         'Packages::Rubygems::ExtractionWorker' => 3,
         'PagesDomainSslRenewalWorker' => 3,
         'PagesDomainVerificationWorker' => 3,
-        'PagesRemoveWorker' => 3,
         'PagesTransferWorker' => 3,
         'PagesUpdateConfigurationWorker' => 3,
         'PagesWorker' => 3,

spec/workers/pages_remove_worker_spec.rb

-# frozen_string_literal: true
-
-require 'spec_helper'
-
-RSpec.describe PagesRemoveWorker do
-  it 'does not raise error' do
-    expect do
-      described_class.new.perform(create(:project).id)
-    end.not_to raise_error
-  end
-end