[GODRIVER-2292] Update load balancer tests to support dedicated load balancer port Created: 31/Jan/22 Updated: 13/Apr/22 Resolved: 17/Feb/22 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | PM Bot | Assignee: | Matt Dale |
| Resolution: | Done | Votes: | 0 |
| Labels: | greenerbuild | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Server Compat: | 5.0, 5.2 | ||||||||||||||||||||||||||||
| Quarter: | FY23Q1 | ||||||||||||||||||||||||||||
| Upstream Changes Summary: | DRIVERS-1983:
See: https://github.com/mongodb-labs/drivers-evergreen-tools/pull/183 |
||||||||||||||||||||||||||||
| Description |
|
This ticket was split from DRIVERS-1983, please see that ticket for a detailed description. |
| Comments |
| Comment by Githook User [ 01/Mar/22 ] |
|
Author: {'name': 'Matt Dale', 'email': '9760375+matthewdale@users.noreply.github.com', 'username': 'matthewdale'}Message: |
| Comment by Githook User [ 01/Mar/22 ] |
|
Author: {'name': 'Matt Dale', 'email': '9760375+matthewdale@users.noreply.github.com', 'username': 'matthewdale'}Message: |
| Comment by Matt Dale [ 17/Feb/22 ] |
|
Fixing these tests actually required fixing:
The load balancer tests pass now (tests impacted by in-progress DNS/spec updates are currently skipped and will be fixed with |
| Comment by Githook User [ 17/Feb/22 ] |
|
Author: {'name': 'Matt Dale', 'email': '9760375+matthewdale@users.noreply.github.com', 'username': 'matthewdale'}Message: |
| Comment by Matt Dale [ 15/Feb/22 ] |
|
The requirement
seems incompatible with the current setup of the initial-dns-seedlist-discovery spec tests. The initial-dns-seedlist-discovery spec tests currently require resolving a mongodb+srv:// scheme URI to a list of hostports, all of which point directly to a local mongos instance (localhost:27017 or localhost:27018). However, to get the new behavior behavior where the server returns a serviceID, the driver needs to connect to the HAProxy hostport (localhost:8000 or localhost:8001 if using run-load-balancer.sh from drivers-evergreen-tools). As a result, the initial-dns-seedlist-discovery spec tests fail because the server doesn't return a seriviceID in the "hello" response description, leading to driver errors because the server doesn't appear to be load-balanced but the driver is configured in load-balancer mode. Previously the Go driver "mocked" the service ID when running a test with "?loadBalanced=true" in the URI. Now that the "mocking" is removed, the tests fail because the server doesn't return a serviceID in "hello" responses if you connect directly to the mongos. Possible fixes: |