-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
Change out Supervisor for better init system #15
Comments
This is a very good idea. |
|
All my opinion, completely open to debate: ➡️ I haven't used S6 personally but know it's popular, so that's likely a good contender. ➡️ Swoole I wouldn't necessarily make as the default as its downsides (long running process that can lead to memory leaks in code - I'm sure a TON of code bases would have this issue as it's not the expected behavior of PHP, unlike say NodeJS, Golang, and just about everything else 😂). Maybe swoole can be another variant vs php-fpm? |
Absolutely. I would definitely offer Swoole as it's own variant. Thanks for the heads up on Swoole. I haven't used it before, so I will definitely looking out for this as I start to toy around with it. I will do some playing around with S6 and keep ya'll posted! 🤠 |
Goodbye Supervisor, hello S6 Overlay! I just merged the S6 enhancements into I spent a lot of time toying with S6 Overlay and I feel good about where it is at now. There still (and always will be) some improvements along the way, but I have been running these images in production with a few of our apps and it's been working great! If you're interested, check it out and let me know your feedback. Thanks for chiming in guys! 🙌 |
Background
fpm-apache
andfpm-nginx
images use Supervisor to bring things upPossible Solution
More details
Hesitations
The text was updated successfully, but these errors were encountered: