Skip to content

xds: add config for ring_hash LB policy extension#20423

Merged
mattklein123 merged 3 commits intoenvoyproxy:mainfrom
markdroth:xds_lb_ring_hash_extension
Mar 21, 2022
Merged

xds: add config for ring_hash LB policy extension#20423
mattklein123 merged 3 commits intoenvoyproxy:mainfrom
markdroth:xds_lb_ring_hash_extension

Conversation

@markdroth
Copy link
Copy Markdown
Contributor

Signed-off-by: Mark D. Roth roth@google.com

Commit Message: xds: add config for ring_hash LB policy extension
Additional Description: Adds extension config for RING_HASH policy, similar to what was added for ROUND_ROBIN in #19517.
Risk Level: Low
Testing: N/A
Docs Changes: Included in PR
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Mark D. Roth <roth@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @mattklein123
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #20423 was opened by markdroth.

see: more, trace.

@markdroth
Copy link
Copy Markdown
Contributor Author

Signed-off-by: Mark D. Roth <roth@google.com>
mattklein123
mattklein123 previously approved these changes Mar 21, 2022
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@mattklein123
Copy link
Copy Markdown
Member

I guess needs format fix.

/wait

Signed-off-by: Mark D. Roth <roth@google.com>
@mattklein123 mattklein123 merged commit 815d447 into envoyproxy:main Mar 21, 2022
@markdroth markdroth deleted the xds_lb_ring_hash_extension branch March 21, 2022 19:21
ravenblackx pushed a commit to ravenblackx/envoy that referenced this pull request Jun 8, 2022
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