openbridge / ob_php-fpm

PHP-FPM container for NGINX using Alpine, Opcache, Dynamic Resources, Monitoring, Security
https://www.openbridge.com
MIT License
96 stars 26 forks source link

unknown directive "max_fails=3" #16

Closed Able2Code closed 6 years ago

Able2Code commented 6 years ago

Hi, using the wordpress.yml and changing the env file to mine, nginx container trow this error: unknown directive "max_fails=3" in /etc/nginx/upstream.d/php-fpm.conf

my ENV APP_DOCROOT=/usr/share/nginx/html PHP_FPM_PORT=9000 PHP_START_SERVERS=16 PHP_MIN_SPARE_SERVERS=8 PHP_MAX_SPARE_SERVERS=16 PHP_MEMORY_LIMIT=256 PHP_OPCACHE_MEMORY_CONSUMPTION=96 PHP_MAX_CHILDREN=16 NGINX_DOCROOT=/usr/share/nginx/html NGINX_SERVER_NAME=testserver NGINX_CONFIG=php NGINX_DEV_INSTALL=true

my YML version: '3.1' services: nginx: image: openbridge/nginx container_name: nginx ports:

tspicer commented 6 years ago

Hi, testserver is not a valid name for NGINX. A valid server name is something that resolves to an address, like localhost or apple.com or dell.com. It won't know what to do with something like testserver

tspicer commented 6 years ago

Also, this issue might be better handled in the NGINX project

tspicer commented 6 years ago

Here is the sample config for PHP from the NGINX server. Notice PHP_FPM_UPSTREAM=localhost:9000 is set to the location where you have PHP deployed. NGINX needs to know where PHP is located so without this it will error. Maybe share the ENV you are using for NGINX?

NGINX_SERVER_NAME=localhost
NGINX_APP_PLUGIN=
NGINX_CONFIG=php
NGINX_DEV_INSTALL=true
PHP_FPM_UPSTREAM=localhost:9000
REDIS_UPSTREAM=redis:6379
NGINX_PROXY_UPSTREAM=localhost:8080
NGINX_DOCROOT=/usr/share/nginx/html
NGINX_CDN_HOST=cdn.myhost.com
Able2Code commented 6 years ago

Hi @tspicer, i left those configurations ( PHP_FPM_UPSTREAM , REDIS_UPSTREAM, NGINX_PROXY_UPSTREAM ) because they were the default configurations in the nginx readme, now everything works as expected, thank you!