Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GOAWAY message leads to other Clients/Channels being unresponsive #1683

Open
konnextv opened this issue Apr 15, 2024 · 0 comments
Open

GOAWAY message leads to other Clients/Channels being unresponsive #1683

konnextv opened this issue Apr 15, 2024 · 0 comments

Comments

@konnextv
Copy link

Bug Report

Version

  • tonic@0.11.0
  • tonic-build@0.10.2

Platform

Linux nomad-client-1 5.15.0-102-generic #112-Ubuntu SMP Tue Mar 5 16:49:56 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

Description

In a scenario where a gRPC Server (in my case hosted with ASP.NET Core) is redeployed and the new address is updated I tried

  • switching the Client completely (creating a new client with ::connect and using that on the next request)
  • using balance_channel and updating the only endpoint with Change::Insert and MyDataClient::new(Channel)

What I observe is the following:

  • the connection to the new address succeeds
  • before the next request is made by the application, the old connection receives a GOAWAY
  • the "new" connection becomes unresponsive

I suspect the old Client's Transport influences the new Client with its attached Channel. Maybe this is expected behaviour and you can enlighten me. Maybe we are on to something, I can provide more logs if needed - I tried to summarize them above.

@konnextv konnextv changed the title GOAWAY message leads to death of other Channels GOAWAY message leads to other Clients/Channels being unresponsive Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant