-
Notifications
You must be signed in to change notification settings - Fork 37
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
Health support #47
Comments
This is the integration with MP Health by providing implicit readiness check - ready to receive traffic; liveness Health procedure - to be killed if DOWN. |
…ivex.rxjava2-rxjava-2.2.6 Bump rxjava from 2.2.5 to 2.2.6
This will be just a spec statement regarding the integration with MP Health. |
Initially this was proposed as integration with MP Health by providing implicit checks.
@Emily-Jiang Why leaving the idea? |
I don't have any objections to take it in. We need a PR to update spec and add tcks. Feel free to contribute @danielkec |
Signed-off-by: Daniel Kec <[email protected]>
@Emily-Jiang PR #130 is ready |
Thank you @danielkec for proposing this PR. It serves a good talking point. I took a look at this PR and referred back to this issue. This issue needs a bit more description on describing what behaviour we want to define. What we are going to do with nack? Are we going to consider livessness to be DOWN when there is one channel has issues? At the moment, you introduce an opt out approach, should we use opt-in approaches? @cescoffier can you please comment on this? |
No description provided.
The text was updated successfully, but these errors were encountered: