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

Add gRPC health check service support to HC filter #2898

Open
futangw opened this issue Mar 26, 2018 · 2 comments
Open

Add gRPC health check service support to HC filter #2898

futangw opened this issue Mar 26, 2018 · 2 comments
Labels
enhancement Feature requests. Not bugs or questions. help wanted Needs help!

Comments

@futangw
Copy link

futangw commented Mar 26, 2018

Hi, I'd like to setup the gRPC health check between two envoys, but looks envoy as a server (upstream) does not generate gRPC health check response?

I had client envoy setting this:
health_checks: [{
timeout: 0.5s,
interval: 5s,
unhealthy_threshold: 2,
healthy_threshold: 3,
grpc_health_check: { service_name: "ic" }
}]

and I had the server envoy setting this:
http_filters:
- name: envoy.health_check
config:
pass_through_mode: false
endpoint: /grpc.health.v1.Health/Check

But looks the server envoy recognizes 'this is a health check' because the statistics "http.ingress_http.tracing.health_check" is creasing, but on the client envoy, the cluster.grpc.health_check.failure is creasing too.

The tcpdump of client envoy health check request as below:

image

The tcpdump of server envoy health check response as below:

grpc_health_check_resonse

when I set "pass_through_mode: true", I did see the HC message pass to my application.

the envoy version is:
envoy version: 5a49ba6/1.6.0-dev/Clean/RELEASE

thank you!

@mattklein123
Copy link
Member

That's correct that right now Envoy itself does not support the gRPC health check service for responding to health checks. I'm going to go ahead and mark this as a feature request.

@mattklein123 mattklein123 changed the title Does envoy support gRPC health check reply? Add gRPC health check service support to HC filter Mar 26, 2018
@mattklein123 mattklein123 added enhancement Feature requests. Not bugs or questions. help wanted Needs help! labels Mar 26, 2018
@futangw
Copy link
Author

futangw commented Mar 27, 2018

@mattklein123 Matt, thanks for the clarification and willing to take enhancement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature requests. Not bugs or questions. help wanted Needs help!
Projects
None yet
Development

No branches or pull requests

2 participants