clientv3/integration: fix TestSetEndpointAndPut dial tests #8869
Conversation
Signed-off-by: Gyu-Ho Lee <gyuhox@gmail.com>
| // grpc/transport.ErrConnClosing or grpc/transport.ErrConnDrain on balancer endpoint switch | ||
| if err != nil && | ||
| !strings.Contains(err.Error(), "transport is closing") && | ||
| !strings.Contains(err.Error(), "the server stops accepting new RPCs") { |
There was a problem hiding this comment.
why we might receive a server side error triggered by a endpoint switch?
There was a problem hiding this comment.
Balancer switch might not have completed, so errors return while gRPC drains current connection and creates a new sub connection.
There was a problem hiding this comment.
transport is closing makes sense. i was asking why we can see the server stops.... gRPC client draining connection has nothing to do with the server.
There was a problem hiding this comment.
gRPC client draining connection has nothing to do with the server.
This error message is more about gRPC http/2 client telling the current connection has been drained.
tearDownon current*addrConn(errConnDrain)ac.transport.GracefulClose()that prevents new streams from being created, setting the connection state to drain
So, it is the http2Client that returns ErrStreamDrain error, not from the server.
|
This needs rework after grpc/grpc-go#1844 is merged. |
Fix #8859.