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

Keepalive on grpc connections #411

Merged
merged 1 commit into from
Apr 28, 2023
Merged

Keepalive on grpc connections #411

merged 1 commit into from
Apr 28, 2023

Conversation

LionelJouin
Copy link
Member

Description

Add keepalive setting for grpc connections towards NSP and IPAM improving the time for connection to be re-established after a failure.

https://www.evanjones.ca/tcp-connection-timeouts.html

Issue link

#355

Checklist

  • Purpose
    • Bug fix
    • New functionality
    • Documentation
    • Refactoring
    • CI
  • Test
    • Unit test
    • E2E Test
    • Tested manually
  • Introduce a breaking change
    • Yes (description required)
    • No

@LionelJouin
Copy link
Member Author

{"severity":"error","timestamp":"2023-04-24T16:03:03.865+00:00","service_id":"Meridio-ipam","message":"watchConduitClient.Recv","version":"1.0.0","extra_data":{"class":"ConduitWatcher","error":"rpc error: code = Unavailable desc = closing transport due to: connection error: desc = \"error reading from server: EOF\", received prior goaway: code: ENHANCE_YOUR_CALM, debug data: \"too_many_pings\""}}

Add keepalive setting for grpc connections towards NSP and IPAM
improving the time for connection to be re-established after a failure.
@LionelJouin LionelJouin merged commit 5e6fa58 into master Apr 28, 2023
@LionelJouin LionelJouin deleted the keepalive branch April 28, 2023 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants