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

lazy loadable EDS cluster #15120

Closed
lambdai opened this issue Feb 20, 2021 · 2 comments
Closed

lazy loadable EDS cluster #15120

lambdai opened this issue Feb 20, 2021 · 2 comments
Assignees
Labels
area/xds enhancement Feature requests. Not bugs or questions.

Comments

@lambdai
Copy link
Contributor

lambdai commented Feb 20, 2021

Background:

The control plane may don't have adequate information to predict what cluster Envoy needs. The currently known approach is to including all possible clusters in the CDS response and all these cluster names will be send to EDS server.
The EDS server will keep envoy updated with the latest endpoints in those clusters.

#Problems:

  1. EDS response on the wire is large. Partial will be resolved by LEDS.
  2. Not all clusters are required. One possible reason is that control plane cannot tell which cluster is needed. Another potential reason is that some cluster are supposed to be used sparsely.

Solution

  1. Introduce lazy load EDS cluster. Various laziness strategies will be discussed in design doc. Providing trade offs among control plane availability and resource usage.
  2. Provide ClusterManager futureThreadLocalCluster to request cluster and get notification on the cluster readiness event.
  3. No need to touch the existing control plane!

Related Proposal

  1. LEDS: independently resolve the EDS bytes on the wire. Locality endpoint discovery service (LEDS) #10373
  2. On Demand CDS Generic Dynamic Forward Proxy with On Demand CDS #15026 . Probably the laziest. Require co-op of control plane. Demand high SLO of control plane.
@lambdai lambdai added enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Feb 20, 2021
@lambdai lambdai self-assigned this Feb 20, 2021
@junr03 junr03 added area/xds and removed triage Issue requires triage labels Feb 22, 2021
@lambdai
Copy link
Contributor Author

lambdai commented Feb 24, 2021

@lambdai
Copy link
Contributor Author

lambdai commented Mar 10, 2021

Close this one in favor of lazy CDS as per community meeting 03/09/2021: https://docs.google.com/document/d/1Yc4zkV-A_cC_R3C0u6D15WQAsRSxADs8p2l8UMMa4P4/edit#

@lambdai lambdai closed this as completed Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/xds enhancement Feature requests. Not bugs or questions.
Projects
None yet
Development

No branches or pull requests

2 participants