feat: Add BackendUtilization + WeightedZones support (WrrLocality Lb policy)#8634
feat: Add BackendUtilization + WeightedZones support (WrrLocality Lb policy)#8634jukie merged 28 commits intoenvoyproxy:mainfrom
Conversation
Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com>
✅ Deploy Preview for cerulean-figolla-1f9435 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
jukie
changed the title
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8634 +/- ##
==========================================
+ Coverage 73.62% 73.64% +0.02%
==========================================
Files 246 246
Lines 49076 49109 +33
==========================================
+ Hits 36131 36167 +36
+ Misses 10923 10919 -4
- Partials 2022 2023 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/retest |
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com>
|
CEL was broken, fixed |
|
/retest |
|
/retest |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3710147722
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
|
/retest |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7d1822a81d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
jukie
force-pushed
the
wrr-locality-support
branch
from
3cd839f to
4a3795c
Compare
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Swish! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
4 participants
What this PR does / why we need it:
Adds the ability to use wrr_locality which is a wrapped Lb Policy of
client_side_weighted_round_robinwhile respecting locality weights.Which issue(s) this PR fixes:
Fixes #8635
Release Notes: No - Existing release note for BackendUtilization is sufficient. I'll follow-up after this with a docs example too.