unitctl: use `restart` instead of `reload` for `apps` subcommand

nginx / unit

NGINX Unit - universal web app server - a lightweight and versatile open source server that simplifies the application stack by natively executing application code across eight different programming language runtimes.

https://unit.nginx.org

Apache License 2.0

5.4k stars 331 forks source link

unitctl: use `restart` instead of `reload` for `apps` subcommand #1426

Closed callahad closed 2 months ago

callahad commented 2 months ago

Bad: unitctl apps reload myapp Good: unitctl apps restart myapp

Why? Control API is /control/applications/myapp/restart

Many other tools use the restart verb

$ docker container restart my_app
$ systemctl restart my_app
$ fly apps restart my_app
$ kubectl rollout restart deployment/my_app