Skip to content

xds: don't allow multiple instances of a locality in the same priority #20574

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.

Sign up for GitHub

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

Merged
adisuissa merged 2 commits into from
Apr 12, 2022
Merged

xds: don't allow multiple instances of a locality in the same priority #20574

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
dfc22ff
xds: don't allow multiple instances of a locality in the same priority
markdroth Mar 29, 2022
eef0ec4
Merge remote-tracking branch 'upstream/main' into eds_duplicate_local…
markdroth Apr 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions api/envoy/config/endpoint/v3/endpoint_components.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,8 @@ message LedsClusterLocalityConfig {
}

// A group of endpoints belonging to a Locality.
// One can have multiple LocalityLbEndpoints for a locality, but this is
// generally only done if the different groups need to have different load
// balancing weights or different priorities.
// One can have multiple LocalityLbEndpoints for a locality, but only if
// they have different priorities.
// [#next-free-field: 9]
message LocalityLbEndpoints {
option (udpa.annotations.versioning).previous_message_type =
Expand Down