Skip to content

Commit 24c7766

Browse files
authored
Merge pull request #5832 from agile6v/healthcheck
fix: remove redundant health check to avoid liveness or readiness timeout
2 parents 5f81b84 + 3844740 commit 24c7766

File tree

1 file changed

+1
-10
lines changed

1 file changed

+1
-10
lines changed

internal/ingress/controller/checker.go

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -61,16 +61,7 @@ func (n *NGINXController) Check(_ *http.Request) error {
6161
return errors.Wrapf(err, "checking for NGINX process with PID %v", pid)
6262
}
6363

64-
statusCode, _, err := nginx.NewGetStatusRequest(nginx.HealthPath)
65-
if err != nil {
66-
return errors.Wrapf(err, "checking if NGINX is running")
67-
}
68-
69-
if statusCode != 200 {
70-
return fmt.Errorf("ingress controller is not healthy (%v)", statusCode)
71-
}
72-
73-
statusCode, _, err = nginx.NewGetStatusRequest("/is-dynamic-lb-initialized")
64+
statusCode, _, err := nginx.NewGetStatusRequest("/is-dynamic-lb-initialized")
7465
if err != nil {
7566
return errors.Wrapf(err, "checking if the dynamic load balancer started")
7667
}

0 commit comments

Comments
 (0)