[SERVER-36986] list_local_sessions.js expects non-existent session to exist when running concurrently with LogicalSessionsCache refresh Created: 04/Sep/18 Updated: 29/Oct/23 Resolved: 25/Sep/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 4.1.2 |
| Fix Version/s: | 3.6.9, 4.0.4, 4.1.4 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Blake Oler | Assignee: | Blake Oler |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Operating System: | ALL | ||||
| Backport Requested: |
v4.0, v3.6
|
||||
| Sprint: | Sharding 2018-09-24 | ||||
| Participants: | |||||
| Description |
InvestigationWhen running a passthrough suite with the logical session cache refresh set to 100ms, it can cause failures in list_local_sessions.js. The test expects a session to exist, but it's likely been reaped by the time that assert.expect is run. Proposed FixAfter some thought, it seems the best idea to blacklist list_sessions.js and list_all_sessions from the logical session cache suites. These tests rely on the command `refreshLogicalSessionCacheNow`, which is supposed to refresh the cache in a deterministic fashion. Running these tests with the background refresh thread on interferes with the deterministic nature of the tests, because refresh operations can happen when they're not supposed to. Since both these tests are in jsCore, I'm not worried about losing test coverage. |
| Comments |
| Comment by Githook User [ 25/Sep/18 ] |
|
Author: {'name': 'Blake Oler', 'email': 'blake.oler@mongodb.com', 'username': 'BlakeIsBlake'}Message: |
| Comment by Blake Oler [ 24/Sep/18 ] |
|
Re-opening because we need to add list_all_local_sessions.js to the blacklists. |
| Comment by Blake Oler [ 21/Sep/18 ] |
|
Author: {'name': 'Blake Oler', 'email': 'blake.oler@mongodb.com', 'username': 'BlakeIsBlake'}Message: |
| Comment by Githook User [ 21/Sep/18 ] |
|
Author: {'name': 'Blake Oler', 'email': 'blake.oler@mongodb.com', 'username': 'BlakeIsBlake'}Message: |
| Comment by Misha Tyulenev [ 20/Sep/18 ] |
|
ack |
| Comment by Blake Oler [ 18/Sep/18 ] |
|
misha.tyulenev proposed fix for you to ack |