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

You need to add the Health.State property for the container #434

Closed
budarin opened this issue May 9, 2022 · 2 comments
Closed

You need to add the Health.State property for the container #434

budarin opened this issue May 9, 2022 · 2 comments

Comments

@budarin
Copy link

budarin commented May 9, 2022

When the service is started, the readiness of the service for operation is checked for some time.
During this time, the service is not ready to accept requests yet, but now the situation is such that we believe that it is already ready and requests that the service cannot serve are pouring in

It is necessary to distinguish between the Health.Status state in order to understand that the server may already be added to the upstream

And also in a situation when the service is overloaded and health check tells us that the service is not healthy - we could exclude it from the upstrem until it becomes operational again

@free6k
Copy link
Contributor

free6k commented Jun 18, 2022

I created pr #442, can check?

@buchdag
Copy link
Member

buchdag commented Dec 19, 2023

Implemented by #442

@buchdag buchdag closed this as completed Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants