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

Handle case where grpc backend may be deregistered from registry but is still handling traffic #912

Open
nathanejohnson opened this issue Nov 18, 2022 · 0 comments

Comments

@nathanejohnson
Copy link
Member

This is an attempt to gracefully shut down a connection to a grpc backend that gets deregistered from a registry, such as consul. This is described as point 1 in #807 . I'm not sure how common this will be, mostly a deregister event is caused by a failing health check, but this is easy enough to implement and shouldn't break anything so worth trying

nathanejohnson added a commit that referenced this issue Nov 18, 2022
…ing GRPC connections.

This should address part 1 of #807 and #912
nathanejohnson added a commit that referenced this issue Nov 18, 2022
…ing GRPC connections.

This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Nov 18, 2022
This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Nov 18, 2022
This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Nov 18, 2022
This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Nov 18, 2022
This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Nov 18, 2022
This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Nov 18, 2022
This should address part 1 of #807 and #912

update grpc docs
nathanejohnson added a commit that referenced this issue Dec 10, 2022
This should address part 1 of #807 and #912

update grpc docs
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

1 participant