-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Labels
Area: Resolvers/BalancersIncludes LB policy & NR APIs, resolver/balancer/picker wrappers, LB policy impls and utilities.Includes LB policy & NR APIs, resolver/balancer/picker wrappers, LB policy impls and utilities.P1Type: Bug
Milestone
Description
What version of gRPC are you using?
1.71.1, master.
What version of Go are you using (go version
)?
go version go1.24.2
What operating system (Linux, Windows, …) and version?
Linux, MacOS
What did you do?
I'm using the least_request_experimental balancer. A resolver reports an error.
What did you expect to see?
Maybe an error log statement reporting the resolver error.
What did you see instead?
gRPC panics because it's embedded balancer at
grpc-go/balancer/leastrequest/leastrequest.go
Lines 100 to 102 in a51009d
// Embeds balancer.Balancer because needs to intercept UpdateClientConnState | |
// to learn about choiceCount. | |
balancer.Balancer |
Calls to methods that are not overridden by leastRequestBalancer
directly should instead be explicitly forwarded to the child balancer (or leastRequestBalancer
should use endpointsharding
as its embedded balancer instead of having a separate child field).
Metadata
Metadata
Assignees
Labels
Area: Resolvers/BalancersIncludes LB policy & NR APIs, resolver/balancer/picker wrappers, LB policy impls and utilities.Includes LB policy & NR APIs, resolver/balancer/picker wrappers, LB policy impls and utilities.P1Type: Bug