httpd icon indicating copy to clipboard operation
httpd copied to clipboard

Make the health check timeout for balancer member configurable

Open Mrida opened this issue 2 years ago • 4 comments
trafficstars

Atfer httpd upgrade to version 55, the health check timeout has been set to be equal to the request timeout. We have lost the capability to define a timeout for the health check that is different from that of the request.

Our worker request timeout is set to 60s (time needed to treat a request) while its health check should be quick (5s). The need for a short health check is to allow httpd to quickly detect suspended workers (that do not receive hc) and set their health status to HcFl so that requests do not get routed to them.

The goal of this pull request is to make the timeout of the health check configurable by balancer member.

Mrida avatar Sep 20 '23 09:09 Mrida

Hello @ylavic, I really appreciate any insights or feedback you might have about this PR.

Mrida avatar Oct 12 '23 09:10 Mrida

Hello @icing, as this PR will be closed soon, I really appreciate any insights or feedback you might have about it.

Mrida avatar Oct 16 '23 08:10 Mrida

@Mrida sorry, I am not really familiar with that part of the code. I recommend contacting the dev mailing list to get more attention at [email protected]

icing avatar Oct 16 '23 08:10 icing

Hello @trawick, I really appreciate any input you might have about this pr.

Mrida avatar Oct 31 '23 10:10 Mrida

UP ! We need this change !

azdfzshffg avatar Dec 20 '24 17:12 azdfzshffg

@ylavic This is a real issue and there are many use cases. I am currently stuck on a project where the application can take a long time to process the request, thus requiring a lengthy BalancerMember timeout. Having a separate healthcheck timeout, like in Nginx or HAproxy, will help without having to rearchitect the infrastructure.

Also mentioned here: https://bz.apache.org/bugzilla/show_bug.cgi?id=60948#c9

magnetarmorbius avatar Mar 06 '25 14:03 magnetarmorbius

I think @ylavic is waiting for the review correction he did.

@Mrida Can you update this change ?

azdfzshffg avatar Jul 23 '25 11:07 azdfzshffg