Skip to content

Commit

Permalink
Merge pull request #3544 from jlledom/fix-sidekiq-rake-task
Browse files Browse the repository at this point in the history
Sidekiq task: remove `index` param
  • Loading branch information
akostadinov authored Sep 12, 2023
2 parents 69d6d0d + d94a471 commit 9a3a04e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion lib/tasks/sidekiq.rake
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ namespace :sidekiq do
envs['RAILS_MAX_THREADS'] = ENV.fetch('RAILS_MAX_THREADS', '1')

args = []
args.push(['--index', ENV.fetch('INDEX', '0')])
args.push(%w[backend_sync billing critical default deletion events low mailers priority web_hooks zync].flat_map { |queue| ['--queue', queue] })

exec(envs, 'sidekiq', *args.flatten)
Expand Down
2 changes: 1 addition & 1 deletion test/unit/tasks/sidekiq_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ class SidekiqTest < ActiveSupport::TestCase
queues = %w[backend_sync billing critical default deletion events low mailers priority web_hooks zync]
.flat_map { |queue| ['--queue', queue] }

Object.any_instance.expects(:exec).with({ 'RAILS_MAX_THREADS' => '1' }, 'sidekiq', '--index', '0', *queues)
Object.any_instance.expects(:exec).with({ 'RAILS_MAX_THREADS' => '1' }, 'sidekiq', *queues)
execute_rake_task 'sidekiq.rake', 'sidekiq:worker'
end
end
Expand Down

0 comments on commit 9a3a04e

Please sign in to comment.