capistrano-sidekiq icon indicating copy to clipboard operation
capistrano-sidekiq copied to clipboard

Published 20 hours ago • verified publisher icon seuros

--index argument must be removed from the sidekiq start script

Open dmitry opened this issue 5 years ago • 6 comments

As of sidekiq 6.0 daemonization removal (https://github.com/mperham/sidekiq/commit/5987a8c352a220a94e1850c79186d82c35050d85#diff-411201f0104d91346964914342a07cefL324) --index argument is not available anymore. If you try to deploy sidekiq 6.0 with --index the deployment will be failed with invalid option: --index.

By the way this argument haven't been used for a long time already (from the removed source code):

this index remains here for backwards compatibility but none of the Sidekiq family use this value anymore. it was used by Pro's original reliable_fetch.

dmitry avatar Sep 11 '19 11:09 dmitry

faced with same issue. @dmitry, how did you manage it?

Senegal avatar Sep 22 '19 05:09 Senegal

@Senegal at the moment downgraded to the sidekiq 5.2 It is not enough just to remove --index as sidekiq 6.0 removed daemonization from it's core and now daemonization should be done by yourself.

dmitry avatar Sep 22 '19 16:09 dmitry

Any update on this issue ? Thank you

guillaumebriday avatar Dec 15 '19 00:12 guillaumebriday

can you try #236 ?
I tested it with a deployment using systemd.

seuros avatar Jan 16 '20 18:01 seuros

Looks, the problem is still actual, any news to merge #236 ?

rusikf avatar Jan 29 '20 09:01 rusikf

@rusikf would be great if you try and write some feedback to #236.

dmitry avatar Feb 04 '20 08:02 dmitry