[GODRIVER-1298] Panic in topology/pool.go:416 Created: 04/Sep/19 Updated: 28/Oct/23 Resolved: 12/Sep/19 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | Core API |
| Affects Version/s: | None |
| Fix Version/s: | 1.1.2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Bradley Wilson-Hunt | Assignee: | Isabella Siu (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Case: | (copied to CRM) | ||||||||
| Description |
|
During some investigation on another issue we were testing the current master branch (22646d953d8106e567b1da9aab98b627a2fb204f) with some of our loadtests and we receive panics from here: https://github.com/mongodb/mongo-go-driver/blob/master/x/mongo/driver/topology/pool.go#L416
It happens consistently accross all the pods.
This is the mongo connection string used: ``` mongodb://xx:xxxx@xxx:27017,xxx:27017,xxxx:27017/matchmaking-service?ssl=true&authSource=admin&maxPoolSize=100&minPoolSize=10 ```
|
| Comments |
| Comment by Bradley Wilson-Hunt [ 16/Sep/19 ] |
|
I can confirm pulling from master that this fix is working! Thanks! |
| Comment by Githook User [ 12/Sep/19 ] |
|
Author: {'username': 'iwysiu', 'email': 'isabella.siu@10gen.com', 'name': 'iwysiu'}Message: Change-Id: I3ae9b0aca1c40c4a1ee3a672b249fabc4ec8b3ad |
| Comment by Githook User [ 12/Sep/19 ] |
|
Author: {'name': 'iwysiu', 'username': 'iwysiu', 'email': 'isabella.siu@10gen.com'}Message: Change-Id: I3ae9b0aca1c40c4a1ee3a672b249fabc4ec8b3ad |
| Comment by Isabella Siu (Inactive) [ 06/Sep/19 ] |
|
code review url: https://review.gerrithub.io/c/mongodb/mongo-go-driver/+/467710 |
| Comment by Bradley Wilson-Hunt [ 05/Sep/19 ] |
|
Im also able to reproduce this same issue in the latest release/1.1 branch (4028be8728b2b5ab8b02c9c5660eff657280f0a3)
If minPoolSize is removed from the uri i can't replicate the issue. However when minPoolSize is removed the application can't handle that many RPS and ends up with even more ctx cancelled. |
| Comment by Bradley Wilson-Hunt [ 04/Sep/19 ] |
|
I appologize for the formatting but it does not seem like i can edit the issue after its been created.. |