Instead of a USR2 we should do a full restart of puma
Currently when there is a configuration or version change, omnibus will issue a unicorn/puma hup or usr2 during reconfigure.
With #4756 (closed) would would like to introduce a uniform command for reloading unicorn and puma.
With the recent introduction of the blackout period, the /-/readiness
endpoint will serve 503s for a short window of time on a reload (USR2 for puma, HUP to the wrapper for unicorn) while still being able to service requests normally.
We would like to optionally, or by default, issue full restarts of puma on reconfigure. This might help with the recent deadlock issue we saw on a single canary server gitlab#118839 (closed) and also would be a lot cleaner for us in general during deploys.
The way to do this currently would be with a gitlab-ctl int puma
instead of gitlab-ctl usr2 puma
. The former will stop the process instead of a re-exec'ing the binary.