You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Not all of h-periodic's beat processes are needed in every environment that h-periodic is deployed to. For example the ca-central-1 deployment of h-periodic doesn't need the checkmate_beat process.
h-periodic currently uses a Python hack to avoid starting the checkmate_beat process if the DISABLE_CHECKMATE_BEAT envvar is set:
I think Supervisor may even restart these processes when they exit! Though I think by default it tries a fixed number of times then gives up, it doesn't keep restarting them forever. I think the restarting can be completely disabled in supervisord.conf but that's not ideal: when a process is actually supposed to be running then we actually do want Supervisor to restart it if it crashes.
The right way to do this is by using envvars to control each process's autostart setting in supervisord.conf, as has been done in h:
Not all of h-periodic's beat processes are needed in every environment that h-periodic is deployed to. For example the
ca-central-1
deployment ofh-periodic
doesn't need thecheckmate_beat
process.h-periodic currently uses a Python hack to avoid starting the
checkmate_beat
process if theDISABLE_CHECKMATE_BEAT
envvar is set:h-periodic/h_periodic/checkmate_beat.py
Lines 12 to 14 in 8aac496
The h_beat and lms_beat have the same hack.
I think Supervisor may even restart these processes when they exit! Though I think by default it tries a fixed number of times then gives up, it doesn't keep restarting them forever. I think the restarting can be completely disabled in
supervisord.conf
but that's not ideal: when a process is actually supposed to be running then we actually do want Supervisor to restart it if it crashes.The right way to do this is by using envvars to control each process's
autostart
setting insupervisord.conf
, as has been done in h:The text was updated successfully, but these errors were encountered: