Skip to content
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

Bug 1823950: [baremetal] Switch to /readyz for haproxy healthchecking #1724

Merged
merged 1 commit into from
May 14, 2020

Commits on May 11, 2020

  1. Switch to /readyz for haproxy healthchecking

    Per [0], the /readyz endpoint is how the api communicates that it
    is gracefully shutting down. Once /readyz starts to report failure,
    we want to stop sending traffic to that backend. If we wait for
    /healthz, it may be too late because once /healthz starts failing
    the api is already not accepting connections.
    
    I also moved the liveness probe for haproxy itself to use a /readyz
    endpoint for consistency. This isn't strictly necessary, but I think
    it will be less confusing if there aren't multiple health check
    endpoints in the config.
    
    0: openshift/installer#3537
    cybertron committed May 11, 2020
    Configuration menu
    Copy the full SHA
    022933c View commit details
    Browse the repository at this point in the history