Skip to content
Snippets Groups Projects
Unverified Commit 5db79829 authored by Muhammad Moinur Rahman's avatar Muhammad Moinur Rahman
Browse files

www/py-gunicorn: Record child process

add -p to daemon to store child process pid

Currently the installed gunicorn rc.d script records the PID of the
supervisor process to /var/run/gunicorn. This is done by passing "-P"
via daemon(8).

Record the primary gunicorn worker process in /var/run as well, this
will enable better integration with tools like newsyslog(8). For example
gunicorn accepts a SIG1 signal to the main worker process to close and
reopen logfiles after rotation to the main worker as well as child
process. By also passing "-p" to daemon we can then have newsyslog get
the appropriate pid
to sent a SIG1 to.

Here are the docs on the signals gunicorn accepts, as there are other
interesting usecases this would enable as well:
https://docs.gunicorn.org/en/stable/signals.html

PR:		281005
Reported by:	pete@nomadlogic.org
parent 423c9569
No related branches found
No related tags found
No related merge requests found
PORTNAME= gunicorn
DISTVERSION= 23.0.0
PORTREVISION= 1
CATEGORIES= www python
MASTER_SITES= PYPI
PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX}
......
......@@ -25,14 +25,16 @@ load_rc_config $name
: ${gunicorn_group=%%GROUPS%%}
pidfile="/var/run/${name}.pid"
cpidfile="/var/run/${name}-worker.pid"
command=/usr/sbin/daemon
command_args="-P ${pidfile} -f -ST ${name} %%PREFIX%%/bin/gunicorn -c ${gunicorn_config}"
command_args="-P ${pidfile} -p ${cpidfile} -f -ST ${name} %%PREFIX%%/bin/gunicorn -c ${gunicorn_config}"
required_files="${gunicorn_config}"
start_precmd="gunicorn_precmd"
gunicorn_precmd()
{
install -o ${gunicorn_user} /dev/null ${pidfile}
install -o ${gunicorn_user} /dev/null ${cpidfile}
}
run_rc_command "$1"
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment