search

shopinvader / docker-locomotive-shopinvader

Run your ShopInvader server in docker !
4 stars 14 forks source link

Fix handling of PUMA_MAX_RAM env var #24

Closed sbidoul closed 1 month ago

sbidoul commented 4 years ago

When the PUMA_MAX_RAM environment variable is set, PumaWorkerKiller was apaprently mostly disabled: the typicall recurring message PumaWorkerKiller: Consuming 338.73828125 mb with master and 2 workers.. was not present in the log.

Converting that variable to an Integer before passing it to the PumaWorkerKiller configuration seems to fix this.

cc/ @Cedric-Pigeon

sbidoul commented 4 years ago

Note this trick does not seem to be necessary for other puma configurations, such as PUMA_WORKERS etc.