connect: built-in proxy/SDK should use ?cached service discovery
Open
theme/connect
type/enhancement
- Dominant language
- Go
- Stars
- 30.1k
- Forks
- 4.6k
- Avg merge
- 2d 6h
- Merged PRs (30d)
- 43
Description
In 1.3.0 we added `?cached` queries for service discovery. The whole reason for adding these was to enable built-in proxy (and other proxy implementations) to "fail static" such that they can still make outbound requests during a partition from the servers or server outage.
Envoy is unaffected since it requires that we "push" the results to it so it's inherently static, Using `?cached` in the SDK is a trivial change that will let built-in proxy and native apps have the same behaviour without having to support full load balancing decisions locally.
Contributor guide
Assessment
This issue has not been assessed yet.