[SERVER-72721] Race in TicketHolderTests Created: 11/Jan/23 Updated: 27/Oct/23 Resolved: 17/Jan/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Gregory Wlodarek | Assignee: | Backlog - Storage Execution Team |
| Resolution: | Gone away | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
|||||||||||||||
| Assigned Teams: |
Storage Execution
|
|||||||||||||||
| Operating System: | ALL | |||||||||||||||
| Steps To Reproduce: |
|
|||||||||||||||
| Participants: | ||||||||||||||||
| Linked BF Score: | 60 | |||||||||||||||
| Description |
|
Multiple tests in the TicketHolder test suite can fail when adding a sleep between the time _enqueuedElements and the _queueMutex is acquired (see linked BFs). It is not safe to rely on _enqueuedElements to determine when an operation has successfully been enqueued. The way the TicketHolder is tested should be changed to account for this. Original Description: |
| Comments |
| Comment by Haley Connelly [ 17/Jan/23 ] |
|
With the change of the TicketHolder implementation in I believe this race, the race between updating the queued() count before actually registering an operation as waiting, has gone away with changes in |