-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Readiness probe error not handled due to php-fpm-healthcheck timeout #41
Comments
I'm getting the same - makes it a little useless when PHP-FPM stops responding |
Just to update with this one for anyone else searching, I managed to resolve using BusyBox livenessProbe:
exec:
command:
- timeout
- "1"
- php-fpm-healthcheck
- --listen-queue=15 # fails if there are more than 10 processes waiting in the fpm queue This means that if php-fpm-healthcheck doesn't return success within 1 second it'll assume its dead. |
Did the same before. It works |
@xriser do you find PHP-FPM becomes unresponsive often? I'm having problems where the node will restart about 10 times a day due to PHP-FPM not responding |
no, it depends on the application. If the app doesn't free php child they start to grow and running out of active child limits. |
If we implement a |
Is this doc relevant? But yes, I think we must implement a |
When php-fpm stuck and didn't respond, php-fpm-healthckeck can't handle this, and timeouts to connect to the port.
root@deployment-php-5bddfd7964-7ctmf:/app# php-fpm-healthcheck -v Trying to connect to php-fpm via: localhost:9000/status ^C
So, php-fpm-healthcheck doesn't return fail status in a reasonable time.
Since K8s doesn't handle timeouts healthcheck, it doesn't work. Because Kubernetes remove the pod from service only when heathcheck returning fail event.
Readiness probe errored: rpc error: code = DeadlineExceeded desc = context deadline exceeded
Readiness probe errored, but pod didn't remove from services.
Is there any way to setup a reasonable timeout to connect to the PHP port for the php-fpm-healthcheck?
The text was updated successfully, but these errors were encountered: