Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
dmitry opened this issue Sep 11, 2019 · 6 comments
Open

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

dmitry opened this issue Sep 11, 2019 · 6 comments

Comments

@dmitry
Copy link

dmitry commented Sep 11, 2019

As of sidekiq 6.0 daemonization removal (sidekiq/sidekiq@5987a8c#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.

@Senegal
Copy link

Senegal commented Sep 22, 2019

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

@dmitry
Copy link
Author

dmitry commented Sep 22, 2019

@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.

@guillaumebriday
Copy link

Any update on this issue ? Thank you

@seuros seuros mentioned this issue Jan 16, 2020
@seuros
Copy link
Owner

seuros commented Jan 16, 2020

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

@rusikf
Copy link

rusikf commented Jan 29, 2020

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

@dmitry
Copy link
Author

dmitry commented Feb 4, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants