consul
consul copied to clipboard
Support Envoy's MaxEjectionPercent and BaseEjectionTime config entries for passive health checks
Description
WAN federated clusters are not able to failover all traffic from one datacenter to another via passive healthchecks when the control plane is down without being able to set MaxEjectionPercent to 100%. BaseEjectionTime just provides additional configurability on the backoff interval until the max ejection interval is hit.
Testing & Reproduction steps
- Existing passive health check unit tests updated
- Existing passive health check integration tests updated to verify configuration
- Open to adding a functional integration test if it would be useful, however I'm planning to add support for these config values via K8s CRDs in the consul-k8s repo. e2e tests on K8s might be more useful.
PR Checklist
- [x] updated test coverage
- [x] external facing docs updated
- [x] not a security concern
@analogue I assume a consul-k8s crd update is also in order?
@analogue I assume a consul-k8s crd update is also in order?
Yep, that is coming up next 😄.
This pull request has been automatically flagged for inactivity because it has not been acted upon in the last 60 days. It will be closed if no new activity occurs in the next 30 days. Please feel free to re-open to resurrect the change if you feel this has happened by mistake. Thank you for your contributions.
Added backport/1.15 since 1.15 is now out.
Looks like this did not backport at all for 1.14 and 1.15.
Thank you for this feature but can we also document it on https://developer.hashicorp.com/consul/docs/v1.14.x/connect/proxies/envoy#passive_health_check?
Thanks!