load_balancing: add load-aware locality LB policy proto, docs, and config stub#44341
Open
jukie wants to merge 5 commits intoenvoyproxy:mainfrom
Open
load_balancing: add load-aware locality LB policy proto, docs, and config stub#44341jukie wants to merge 5 commits intoenvoyproxy:mainfrom
jukie wants to merge 5 commits intoenvoyproxy:mainfrom
Conversation
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com>
jukie
changed the title
|
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
Contributor
Author
|
CC @tonya11en |
jukie
commented
Apr 9, 2026
Comment on lines
+232
to
+238
| 1. For each locality, compute the average utilization of its endpoints from their | ||
| most recent ORCA reports. The metric precedence order is: | ||
|
|
||
| a. The max of any metrics listed in ``metric_names_for_computing_utilization`` | ||
| that are present in the report (e.g. ``named_metrics.foo``), if configured. | ||
| b. ``application_utilization`` if reported and > 0. | ||
| c. ``cpu_utilization``. |
Contributor
Author
There was a problem hiding this comment.
Will update this to align with #44196 once that PR is merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Commit Message: load_balancing: add load-aware locality LB policy proto, docs, and config stub
Additional Description: Splitting #43784 as requested and this adds the proto and docs for the new LB policy.
Risk Level: Low
Testing: Config validation unit test
Docs Changes: Added arch overview page for load-aware locality LB policy
Release Notes: added new extension
envoy.load_balancing_policies.load_aware_localityPlatform Specific Features: n/a
xRef #43665
API Considerations:
LoadAwareLocalitywith fields for EWMA tuning, variance threshold, probe percentage, weight expiration, and configurable ORCA metric names.endpoint_picking_policyaccepts any endpoint-picking child LB policy.AI was used during implementation but I fully understand the changes.