Merge branch 'mg-remove-queue-selector-and-negate-options' into 'master'

Remove queue_selector and negate options from Sidekiq

See merge request https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/7540

Merged-by: Balasankar 'Balu' C <[email protected]>
Approved-by: Andrew Patterson <[email protected]>
Approved-by: Balasankar 'Balu' C <[email protected]>
Co-authored-by: Gregorius Marco <[email protected]>

Author: balasankarc

files/gitlab-cookbooks/gitlab/attributes/default.rb

@@ -727,12 +727,10 @@
 default['gitlab']['sidekiq']['health_checks_listen_port'] = 8092
 
 # Cluster specific settings
-default['gitlab']['sidekiq']['queue_selector'] = false
 default['gitlab']['sidekiq']['interval'] = nil
 default['gitlab']['sidekiq']['concurrency'] = nil
 default['gitlab']['sidekiq']['max_concurrency'] = 20
 default['gitlab']['sidekiq']['min_concurrency'] = nil
-default['gitlab']['sidekiq']['negate'] = false
 default['gitlab']['sidekiq']['queue_groups'] = ['*']
 default['gitlab']['sidekiq']['consul_service_name'] = 'sidekiq'
 default['gitlab']['sidekiq']['consul_service_meta'] = nil

files/gitlab-cookbooks/gitlab/templates/default/sv-sidekiq-run.erb

@@ -28,17 +28,11 @@ exec chpst -e /opt/gitlab/etc/gitlab-rails/env -P \
       <% if node['gitlab']['sidekiq']['min_concurrency'] %>
       --min-concurrency <%= node['gitlab']['sidekiq']['min_concurrency'] %> \
       <% end %>
-      <% if node['gitlab']['sidekiq']['queue_selector'] %>
-      --queue-selector \
-      <% end %>
       <% if node['gitlab']['sidekiq']['shutdown_timeout'] %>
       --timeout <%= node['gitlab']['sidekiq']['shutdown_timeout'] %> \
       <% end %>
       <% node['gitlab']['sidekiq']['queue_groups'].each do |queue| %>
         "<%= queue %>" \
       <% end %>
-      <% if node['gitlab']['sidekiq']['negate'] %>
-      --negate \
-      <% end %>
 
 # Do not remove this line; it prevents trouble with the trailing backslashes above.

files/gitlab-cookbooks/package/libraries/deprecations.rb

@@ -328,14 +328,14 @@ def list(existing_config = nil)
           },
           {
             config_keys: %w(gitlab sidekiq queue_selector),
-            deprecation: '15.9',
-            removal: '17.0',
+            deprecation: '15.9', # Remove message issue: https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/8495
+            removal: '17.0', # Removal issue: https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/2220
             note: "Starting with GitLab 17.0, running Sidekiq with queue selector (`sidekiq['queue_selector'] = true`) will be removed. We recommend to follow the steps at https://docs.gitlab.com/ee/administration/sidekiq/extra_sidekiq_processes.html#start-multiple-processes, to run Sidekiq with multiple processes while listening to all queues."
           },
           {
             config_keys: %w(gitlab sidekiq negate),
-            deprecation: '15.9',
-            removal: '17.0',
+            deprecation: '15.9', # Remove message issue: https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/8495
+            removal: '17.0', # Removal issue: https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/2220
             note: "Starting with GitLab 17.0, running Sidekiq with negate (`sidekiq['negate'] = true`) will be removed. We recommend to follow the steps at https://docs.gitlab.com/ee/administration/sidekiq/extra_sidekiq_processes.html#start-multiple-processes, to run Sidekiq with multiple processes while listening to all queues."
           },
           {