Arjan Singh Bal

I think the test is flaking due to the race mentioned in the https://github.com/grpc/grpc-go/pull/8217#discussion_r2025360455, if listener restarts before the ADS stream completes the backoff, the channel doesn't go into TF...

To repro, we can add a 10 millis sleep b/w the listener being stopped and restarted here: https://github.com/grpc/grpc-go/blob/ec4810caeb8c0690cbca03d676d7e523bbec4204/xds/internal/xdsclient/metrics_test.go#L313-L318 This means that the test assumption that a server failure metric is...

I attempted to repro this situation in a unit test by having two clusters with 2 endpoints each. I specified a weighted clusters route action that splits traffic evenly b/w...

I see the problem, in your [test client](https://github.com/axkum10/gRPC_xDS/blob/81860a9487868f4a2db32aff0590a1de2d0aec0c/Service_Discovery/app/src/client.go#L52C77-L52C100), you've set `grpc.WaitForReady(true)`. This causes the current RPC call to block until the server is up. You would need to either set...

Since the decision to send requests to a cluster with only failing subchannels is a cross-language behaviour, not something specific to grpc-go, asking @markdroth to chime in. Mark, can you...

> What is the panic, exactly? There's no explanation in the PR description and no linked issue here. Attached logs to the PR description. The problem is that the child...

> Is this just covering up for a child LB policy that isn't following the rules of the API? Should the child policy be required to set its state? Are...

@dfawley I've changed the approach to ignore `nil` child pickers instead of assuming child LB policies start in `CONNECTING`. The PR description and title are updated to reflect the same.

Closing this PR, will open it once https://github.com/grpc/grpc-go/issues/7686 is fixed.

Hi @peterchenadded, there isn't a way to selectively disable service config lookups for hosts. You will need to use the `WithDisableServiceConfig()` dial option or provide an IP address as the...