-
Notifications
You must be signed in to change notification settings - Fork 695
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
Reloading configuration without restart #55
Comments
This sounds like something that might extend the ManagementFlag interface? http://projects.unbit.it/uwsgi/wiki/ManagementFlag (Yes, yes, I know, sorry -- ManagementFlag hasn't been ported to the new docs yet :D) |
i think the whole management flag subsystem should be rewritten to support more (all ?) parameters. Example: {'max-requests':'1000', 'harakiri':'40'} => will update both max-requests and harakiri without taking down the instance while {'sockets':'/tmp/foobar'} => is not an onthefly option and it will be ignored |
Adding possibility to update all settings on-the-fly would require massive changes to whole code base, and forkish nature of uWSGI will make it tricky. I've added it to 2.0 milestone, IMHO we should try to come up with a better solution for this during 2.0 development (when there is possibility to make backward incompatible changes if needed) and once there is a way of doing this in the core, we can start migrating options to it. |
I think this will be pretty undoable without a complete rewrite, and, in addition to this, will make plugin development really complex (as you need to manage locking when reading options). Closing it, maybe some idea will popup in the future. |
Right now any change in vassal config will cause its restart but some options could be changed without that, for example
--max-requests
or--cron
. I guess that this would require big changes in options handling (like addingrequires restart
flag to option definition or so) and I don't really need it right away, but it would be great if uWSGI could eventually have such feature.The text was updated successfully, but these errors were encountered: