application-gateway-kubernetes-ingress icon indicating copy to clipboard operation
application-gateway-kubernetes-ingress copied to clipboard

new annotation for custom health probe path

Open darrenk13 opened this issue 4 years ago • 2 comments

Is your feature request related to a problem? Please describe. Currently the health probe path is obtained from the readiness probes of the pods in the deployment setup behind the ingress controller. With Istio, the readiness probe is on port 15020 with a path of "/healthz/ready". AGIC picks up the path from this and sends the health checks to the traffic port which then goes to the service that Istio routes traffic to. In my case, that path is invalid and will return a 404 which will fail the health check on the application gateway. The best solution would be if the application gateway supported health checks on a different port from the traffic port but since it doesn't, having the ability to set a custom health probe path will help make things easier.

Describe the solution you'd like I would like a new annotation created with a suggested name of "appgw.ingress.kubernetes.io/backend-probe-path" where a custom probe path could be defined on the listener.

darrenk13 avatar Jun 09 '20 23:06 darrenk13

Hello @darrenk13

I think this annotation solves your issue:

https://github.com/Azure/application-gateway-kubernetes-ingress/blob/master/docs/annotations.md#health-probe-path

Can you please confirm ?

thanks

zioproto avatar Jul 18 '22 17:07 zioproto

Hi @zioproto I'm also a subscriber to this issue - neither Darren nor I work for the company where we saw this issue anymore, so unfortunately Darren probably won't be able to confirm whether this fixes the specific problem we were having. I would say it looks like it should if my memory of the problem is accurate.

Thanks!

gchappel avatar Aug 02 '22 10:08 gchappel