DevFW-CICD/ingress-nginx-helm
14
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #5832 from agile6v/healthcheck

Browse source 
fix: remove redundant health check to avoid liveness or readiness timeout
This commit is contained in:
Kubernetes Prow Robot 2020-07-02 17:34:48 -07:00 committed by GitHub
commit 24c7766d7c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
internal/ingress/controller/checker.go
View file

@ -61,16 +61,7 @@ func (n *NGINXController) Check(_ *http.Request) error {
return errors.Wrapf(err, "checking for NGINX process with PID %v", pid) return errors.Wrapf(err, "checking for NGINX process with PID %v", pid)
} }
statusCode, _, err := nginx.NewGetStatusRequest(nginx.HealthPath) statusCode, _, err := nginx.NewGetStatusRequest("/is-dynamic-lb-initialized")
if err != nil {
return errors.Wrapf(err, "checking if NGINX is running")
}
if statusCode != 200 {
return fmt.Errorf("ingress controller is not healthy (%v)", statusCode)
}
statusCode, _, err = nginx.NewGetStatusRequest("/is-dynamic-lb-initialized")
if err != nil { if err != nil {
return errors.Wrapf(err, "checking if the dynamic load balancer started") return errors.Wrapf(err, "checking if the dynamic load balancer started")
} }