envoyproxy / envoyproxy/envoy

[CDS] Add/port retry_policy to Envoy cluster.

Open
#25,751 26 comments 0 reactions 1 assignee Claimed by @stevenzzzz View on GitHub
area/router enhancement no stalebot
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 22h
Merged PRs (30d)
430

Description

*Title*: Support retry_policy at Cluster.
*Description*:
> Oftentimes, different clusters have its own retry policies, right now Envoy only support Route based retry policy, which means a user needs to configure every route pointing to that Cluster with exactly the same retry policy. it's an unnecessary waste of RAM considering how "bulky" the retrypolicy proto/impl is. This scenario is oftentimes seen for routes/vhost that points to a external [public] service, e.g. cloud storage.

I think we should also support RetryPolicy on a cluster, and to support finer grained control,
make it configurable if a cluster retry policy, when present, should overwrite the route level retry policy, considering that usually route-entry-level retry policy falls back to vhost level retry policy.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.