Skip to content

grpc-xds/load_balancing: Cluster manager implementation. - #2884

Merged
nathanielford merged 43 commits into
grpc:masterfrom
nathanielford:feature/cluster-manager-lb
Oct 15, 2026
Merged

nathanielford merged 43 commits into
grpc:masterfrom
nathanielford:feature/cluster-manager-lb

Conversation

@nathanielford

@nathanielford nathanielford commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

Implementation of XDS Cluster Manager

Solution

Leverages ChildManager to manage child policies. Carries functionality to parse ClusterManager-specific configuration, do the resolver fanout and picker assemblage and handles the per-rpc routing and enforcement of drops on missing/unknown attributes/clusters.

Notes

Comment thread grpc/src/client/load_balancing/child_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
@nathanielford
nathanielford force-pushed the feature/cluster-manager-lb branch from fdc3ac3 to 38adbed Compare September 24, 2026 21:42
@nathanielford
nathanielford marked this pull request as ready for review September 25, 2026 00:04

@dfawley dfawley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I only reviewed cluster_manager - which mostly LGTM - because I believe Arjan has the full call attributes implementation.

Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
@dfawley dfawley assigned nathanielford and unassigned dfawley Sep 25, 2026
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
Comment thread grpc-xds/Cargo.toml Outdated
@arjan-bal

Copy link
Copy Markdown
Contributor

Please use the format crate-name/mod-name: summary for the title and the A-grpc-next label to the PR.

@arjan-bal arjan-bal removed their assignment Sep 28, 2026
@nathanielford nathanielford changed the title Cluster manager implementation. gprc-xds/load_balancing: Cluster manager implementation. Sep 28, 2026
@arjan-bal arjan-bal removed their assignment Sep 29, 2026
@arjan-bal

Copy link
Copy Markdown
Contributor

Please allow reviewers to resolve their own comments. When comments are resolved preemptively, I have to manually expand each thread to verify the new changes.

@nathanielford nathanielford changed the title gprc-xds/load_balancing: Cluster manager implementation. grpc-xds/load_balancing: Cluster manager implementation. Sep 29, 2026

@arjan-bal arjan-bal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM with a minor comment. Please wait for #2878 to be merged.

Comment thread grpc-xds/src/load_balancing/cluster_manager.rs Outdated
@arjan-bal arjan-bal assigned nathanielford and unassigned arjan-bal Sep 30, 2026
@nathanielford
nathanielford merged commit 3aec51b into grpc:master Oct 15, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants