[GODRIVER-2464] Add timeout for RTT monitor "hello" operations Created: 17/Jun/22  Updated: 28/Oct/23  Resolved: 13/Jul/22

Status: Closed
Project: Go Driver
Component/s: None
Affects Version/s: None
Fix Version/s: 1.10.0

Type: Improvement Priority: Unknown
Reporter: Matt Dale Assignee: Matt Dale
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
related to DRIVERS-2386 Improve reliability of SDAM heartbeat... Implementing
is related to GODRIVER-2438 Pool "closeConnection" method cause p... Closed
Backwards Compatibility: Fully Compatible
Documentation Changes: Not Needed

 Description   

Currently the RTT monitor uses a context without timeout to run the "hello" operation (see here). As a result, it's possible for RTT monitor "hello" operations to hang indefinitely if there is a network interruption, preventing the monitor from recording any RTT samples. The motivation for this ticket comes from troubleshooting GODRIVER-2438.

Add a timeout to the RTT monitor "hello" operation to prevent network issues from causing the RTT monitor to hang.



 Comments   
Comment by Githook User [ 14/Jul/22 ]

Author:

{'name': 'Matt Dale', 'email': '9760375+matthewdale@users.noreply.github.com', 'username': 'matthewdale'}

Message: GODRIVER-2464 Add delay in RTT monitor test so Windows can measure a non-zero latency. (#1022)
Branch: master
https://github.com/mongodb/mongo-go-driver/commit/f9e6b2fb48ad497552bb6800857a5aaaf3e3cb39

Comment by Githook User [ 12/Jul/22 ]

Author:

{'name': 'Matt Dale', 'email': '9760375+matthewdale@users.noreply.github.com', 'username': 'matthewdale'}

Message: GODRIVER-2464 Add timeout for RTT monitor hello operations. (#994)
Branch: master
https://github.com/mongodb/mongo-go-driver/commit/d86e0aafe90e45c7e24585336c092c24e30d34cb

Comment by Matt Dale [ 21/Jun/22 ]

PR: https://github.com/mongodb/mongo-go-driver/pull/994

Generated at Thu Feb 08 08:38:40 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.