Possible regression when using NewClient instead of Dial #7075
Closed
Description
#7029 introduced the NewClient() API which should potentially replace grpc.Dial()
However in this branch, when I updated the orca example to use NewClient I'm seeing an error on the client side.
2024/04/02 13:24:29 Error from UnaryEcho call: rpc error: code = DeadlineExceeded desc = context deadline exceeded
This needs investigation, and possibly prioritization before the next release this week
Activity
arvindbr8 commentedon Apr 2, 2024
this is an issue with the way the orcaLB is setup. The
UpdateClientConnState
method was asserting that the resolver returns only 1 address. this was true in the case ofpassthrough
, however sinceNewClient
uses dns as default schema the resolver returned two addresses in Github VM (and in my local machine) - 1 ipv4 and 1 ipv6 address. This is not a regression.