Enhancement Request: Add "systemctl restart nginx" button about nginx-ui HOT 8 OPEN

Wouldn't nginx reload make more sense? This would make sure you don't end up with a nginx that won't start because of syntax errors.

from nginx-ui.

Hello @LucaNerlich, thank you very much for your issue. You're right, of course. The feature is important and will come. The goal for the next release is to manage multiple nginx instances. Since there are environments where nginx is running on both bare-metal and docker, I am looking for a good solution to cover both. I imagine that when integrating an nginx instance into nginx ui the type of the instance is specified (baremetal/docker).

from nginx-ui.

I've been pitching this tool at my workplace and this is the main feature stopping us from using it right now.

from nginx-ui.

nginx -s reload would be ideal, to make sure nginx falls back if the new configuration fails.

from nginx-ui.

To execute a command on non-dockerized nginx from inside a docker container is going to be tricky / hacky. Im unsure myself, on how a good solution would look like.

My approach would be an SSH connection to the server with the necessary permissions to trigger a reload from nginx. The access data would then have to be stored in the form of a user management system. This would also allow the infrastructure to ensure that everyone is only allowed to do what the admin allows. :)

Best David

from nginx-ui.

Well, ideally youd build the output of ‚nginx -t‘ into the you as well. Or add a button ‚check‘ and restart/reload afterwards.

True, good point.

from nginx-ui.

nginx -s reload would be ideal, to make sure nginx falls back if the new configuration fails.

But for any of this to work the nginx binary will need to be mounted into the container.

from nginx-ui.

To execute a command on non-dockerized nginx from inside a docker container is going to be tricky / hacky. Im unsure myself, on how a good solution would look like.

from nginx-ui.