Skip to content

api: clarify non-empty LDS/CDS resource hints.#7512

Merged
htuch merged 2 commits intoenvoyproxy:masterfrom
htuch:cds-lds-resource-addressing
Jul 12, 2019
Merged

api: clarify non-empty LDS/CDS resource hints.#7512
htuch merged 2 commits intoenvoyproxy:masterfrom
htuch:cds-lds-resource-addressing

Conversation

@htuch
Copy link
Copy Markdown
Member

@htuch htuch commented Jul 9, 2019

In order to better support clients such as gRPC-LB that want to access
only a single listener/cluster, provide the scope in the xDS
specification to specify explicit resource hints.

Signed-off-by: Harvey Tuch htuch@google.com

In order to better support clients such as gRPC-LB that want to access
only a single listener/cluster, provide the scope in the xDS
specification to specify explicit resource hints.

Signed-off-by: Harvey Tuch <htuch@google.com>
@htuch htuch requested a review from mattklein123 July 9, 2019 21:28
@htuch
Copy link
Copy Markdown
Member Author

htuch commented Jul 9, 2019

CC @envoyproxy/api-shepherds @envoyproxy/udpa-wg @markdroth @vishalpowar

@markdroth
Copy link
Copy Markdown
Contributor

LGTM. Thanks!

mattklein123
mattklein123 previously approved these changes Jul 9, 2019
@vishalpowar
Copy link
Copy Markdown
Contributor

Update the discovery.proto ?

// returned. LDS/CDS expect empty resource_names, since this is global

Signed-off-by: Harvey Tuch <htuch@google.com>
@mattklein123 mattklein123 self-assigned this Jul 11, 2019
@htuch htuch merged commit 7ef20d7 into envoyproxy:master Jul 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants