Skip to content

A114: WRR Support for Custom Backend Metrics#536

Open
sauravzg wants to merge 5 commits intogrpc:masterfrom
sauravzg:wrr-custom-metrics
Open

A114: WRR Support for Custom Backend Metrics#536
sauravzg wants to merge 5 commits intogrpc:masterfrom
sauravzg:wrr-custom-metrics

Conversation

@sauravzg
Copy link

This proposal allows configuring the weighted_round_robin policy to use custom backend metrics (via ORCA) for computing endpoint weights, instead of relying solely on standard utilization metrics. This aligns gRPC with xDS custom backend metric support.

@sauravzg sauravzg force-pushed the wrr-custom-metrics branch 3 times, most recently from 1423c95 to 6ed9125 Compare February 3, 2026 11:01
Copy link
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

This looks really good! I have just a few minor comments.

Please let me know if you have any questions. Thanks!

Copy link
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

I'd like @ejona86 and @dfawley to take a quick pass on this before we merge it.

@ejona86 ejona86 self-requested a review March 3, 2026 18:32
Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

Seems pretty close

Copy link
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

Just one minor nit remaining, otherwise looks good from my end!

@sauravzg
Copy link
Author

@ejona86 PTAL if we are okay to move forward with submitting this.

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.

5 participants