Skip to content

Commit cae9ce6

Browse files
marcogregoriusbalasankarc
authored andcommitted
Remove queue_selector and negate options from Sidekiq
https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/2220 Changelog: removed
1 parent 8ea9c2a commit cae9ce6

File tree

3 files changed

+4
-12
lines changed

3 files changed

+4
-12
lines changed

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -727,12 +727,10 @@
727727
default['gitlab']['sidekiq']['health_checks_listen_port'] = 8092
728728

729729
# Cluster specific settings
730-
default['gitlab']['sidekiq']['queue_selector'] = false
731730
default['gitlab']['sidekiq']['interval'] = nil
732731
default['gitlab']['sidekiq']['concurrency'] = nil
733732
default['gitlab']['sidekiq']['max_concurrency'] = 20
734733
default['gitlab']['sidekiq']['min_concurrency'] = nil
735-
default['gitlab']['sidekiq']['negate'] = false
736734
default['gitlab']['sidekiq']['queue_groups'] = ['*']
737735
default['gitlab']['sidekiq']['consul_service_name'] = 'sidekiq'
738736
default['gitlab']['sidekiq']['consul_service_meta'] = nil

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

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,17 +28,11 @@ exec chpst -e /opt/gitlab/etc/gitlab-rails/env -P \
2828
<% if node['gitlab']['sidekiq']['min_concurrency'] %>
2929
--min-concurrency <%= node['gitlab']['sidekiq']['min_concurrency'] %> \
3030
<% end %>
31-
<% if node['gitlab']['sidekiq']['queue_selector'] %>
32-
--queue-selector \
33-
<% end %>
3431
<% if node['gitlab']['sidekiq']['shutdown_timeout'] %>
3532
--timeout <%= node['gitlab']['sidekiq']['shutdown_timeout'] %> \
3633
<% end %>
3734
<% node['gitlab']['sidekiq']['queue_groups'].each do |queue| %>
3835
"<%= queue %>" \
3936
<% end %>
40-
<% if node['gitlab']['sidekiq']['negate'] %>
41-
--negate \
42-
<% end %>
4337

4438
# Do not remove this line; it prevents trouble with the trailing backslashes above.

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -328,14 +328,14 @@ def list(existing_config = nil)
328328
},
329329
{
330330
config_keys: %w(gitlab sidekiq queue_selector),
331-
deprecation: '15.9',
332-
removal: '17.0',
331+
deprecation: '15.9', # Remove message issue: https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/8495
332+
removal: '17.0', # Removal issue: https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/2220
333333
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."
334334
},
335335
{
336336
config_keys: %w(gitlab sidekiq negate),
337-
deprecation: '15.9',
338-
removal: '17.0',
337+
deprecation: '15.9', # Remove message issue: https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/8495
338+
removal: '17.0', # Removal issue: https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/2220
339339
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."
340340
},
341341
{

0 commit comments

Comments
 (0)