[GODRIVER-2276] Require hello command + OP_MSG when 'loadBalanced=True' Created: 20/Jan/22  Updated: 22/Feb/22  Resolved: 22/Feb/22

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

Type: Improvement Priority: Major - P3
Reporter: PM Bot Assignee: Benji Rewis (Inactive)
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on GODRIVER-2292 Update load balancer tests to support... Closed
Issue split
Upstream Changes Summary:

DRIVERS-2033:
This ticket is an add-on to DRIVERS-1916. This ticket similarly specifies drivers must ensure that if load balancing mode is enabled, both "hello" and OP_MSG should be used for the connection handshake.
For an example of what your connection process should look like after this change, take a look at the Python pseudocode implementation in the handshake spec.


 Description   

This ticket was split from DRIVERS-2033, please see that ticket for a detailed description.



 Comments   
Comment by Githook User [ 22/Feb/22 ]

Author:

{'name': 'Benjamin Rewis', 'email': '32186188+benjirewis@users.noreply.github.com', 'username': 'benjirewis'}

Message: GODRIVER-2276 Use OP_MSG and hello with loadBalanced (#847)
Branch: master
https://github.com/mongodb/mongo-go-driver/commit/e7214cc18b5894e9d06112fbcf484a0522802739

Comment by Benji Rewis (Inactive) [ 20/Feb/22 ]

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

Comment by Benji Rewis (Inactive) [ 17/Feb/22 ]

Cool thanks for the note, julius.park. This is still blocked on fixing our load balancer tests in Evergreen (GODRIVER-2292), but I'll get back to this soon.

Comment by Julius Park (Inactive) [ 14/Feb/22 ]

As jeff_y uncovered I neglected to include a

 "minServerVersion": "3.6", 

line for test "connection establishment if loadBalanced is specified for non-load balanced clusters" in non-lb-connection-establishment. This was not needed in PyMongo as we do not support <= MongoDB 3.6, but as the Go driver supports <= 3.6, y'all will run into this issue.

Comment by Benji Rewis (Inactive) [ 02/Feb/22 ]

There's no way to test this functionality in Evergreen until we fix our failing load balanced tests with GODRIVER-2292.

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