-
Notifications
You must be signed in to change notification settings - Fork 44
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
Support Passenger 5 restarting #2
Comments
#4 was merged |
This does not work for me, so I'm not sure if there's still a lingering issue or if something else in my environment is the cause... I'm going to try & give as much info as I can, but I've been working at this for at least an hour so it's likely that I've missed something - let me know if you need more information. Software ruby 2.1.5 config/deploy/production.rb
Output at end of deploy:
It seems to take the config, since this is the command it's running to restart the app:
The error seems to indicate that it's not getting the
If I add that user to the If I try using
If I could get this working via Edit I've just rolled back to One thing I noticed after going back to version 0.0.2 is that version 0.0.3 had stopped working with sidekiq entirely (wasn't stopping anything, wasn't starting anything), and it may have also been skipping the part where it gets rid of old releases (I say 'may have' because I don't recall its earlier behavior, but my 0.0.2 deploy removed a dozen old releases). |
@JamesChevalier What version of passenger are you running? You say that capistrano-passenger 0.0.2 works for you, which would suggest you are using a version of passenger < 5, but capistrano-passenger is detecting version 5. Is rvm installed at the system level or the user level? i.e. What's the output of Is passenger installed via rvm? Run Also, can you set log_level to debug so that we can see the exact command that the As for sidekiq, I have no experience with it, but a quick look at the sourcecode makes me wonder if there's some weird interaction involving
and see what the output is. It should contain passenger and sidekiq among others. |
I'm having the same exact problem and went through just about all the steps that @JamesChevalier did. Reverting to capistrano-passenger (0.0.2) has also solved the problem for the time being. |
@VEkh Can you address the same questions I asked @JamesChevalier so that we can figure this one out? |
Same issue for me - here's the answers to your questions: What version of passenger are you running? i.e. What's the output of cap rvm:hook?
Is passenger installed via rvm? Also, can you set log_level to debug so that we can see the exact command that the passenger:restart task sends to your app server? Sometimes at the debug level the command isn't exactly what it's reported to be at the info level.
|
Ok think I've got this one sorted, it was just a permission issue so you need to update Use
Then in your deploy.rb add
(*) it needs to be the end of the file or after |
Thanks for helping out, @betesh !
|
strange failure with capistrano-passenger (0.0.4):
but the command is fine when I ran it on the server manually. |
@zjx20 I think you're the first to spot why this is happening--@JamesChevalier's output contained the same thing:
And so did @kule's. It looks like we're not really passing the full command. Note that this error message is only triggered when your console session is not a TTY. I need to do some more digging on this to figure out why it's outputting the correct command at INFO level but not sending the full command to the server. |
I'm putting this issue on hold for now, because of phusion/passenger#1392 |
@betesh Incase it's helpful; when I've set this up using sudo the command runs correctly i.e.
Not sure exactly where |
@kule You're on the right track. When the first arg to That's how we're able to configure However, there is currently no way to restart an app using passenger 5 without |
@betesh makes sense thanks for the explanation :) |
@JamesChevalier @zjx20 @kule I think #8 fixed your issue. Still waiting for phusion/passenger#1392 before addressing the sudo issue, but this should get you a step closer, so that @kule's solution of editing visudo will probably work for you now. |
I've just finished implementing a change in Passenger which would allow passenger-config restart-app to work without sudo. It is expected that this change lands in 5.0.10. See phusion/passenger#1392 (comment) |
Aaand it's live: https://blog.phusion.nl/2015/06/11/phusion-passenger-5-0-10/ |
There were a number of bugs in the sudo-less restart mechanism in 5.0.10. They will be addressed in 5.0.11, which will be released soon. Please check the changelog for details: https://github.com/phusion/passenger/blob/stable-5.0/CHANGELOG |
Passenger 5 introduces a new way to restart apps:
passenger-config restart-app
More information over at their blog:
https://blog.phusion.nl/2015/03/04/whats-new-in-passenger-5-part-2-better-logging-better-restarting-better-websockets-and-more/
Would be nice to see this implemented in a nifty way.
(auto detecting Passenger 4 / 5?)
The text was updated successfully, but these errors were encountered: