[SERVER-52867] Make sure secondary has finished closing connections after being removed before sending new commands in awaitable_hello_on_nodes_with_invalid_configs.js Created: 14/Nov/20 Updated: 29/Oct/23 Resolved: 30/Nov/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 4.4.0 |
| Fix Version/s: | 4.9.0, 4.4.4 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Jason Chan | Assignee: | Jason Chan |
| 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.4
|
||||||||
| Sprint: | Repl 2020-11-30, Repl 2020-12-14 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 35 | ||||||||
| Description |
|
Currently, we use assert.soon to poll the secondary using replSetGetStatus to check if the node recognizes that it has been removed via heartbeat reconfig. The test then moves on by issuing commands on the removed secondary. However, it is possible that we poll at this exact moment where the secondary has updated its replSetStatus response and drops its lock, and in that case, it is posslble we start issuing commands on the node when it starts to close all connections as part of transitioning to REMOVED. This causes network errors to happen sporadically in our patch builds in this specific part of the test. I think ideally, we want to wait for the connections to finish closing before we start issuing commands. We can do this by adding a failpoint. |
| Comments |
| Comment by Githook User [ 15/Dec/20 ] |
|
Author: {'name': 'Jason Chan', 'email': 'jason.chan@mongodb.com', 'username': 'jasonjhchan'}Message: (cherry picked from commit c54c110248bca5360db277d4a9f011186d55b396) |
| Comment by Githook User [ 30/Nov/20 ] |
|
Author: {'name': 'Jason Chan', 'email': 'jason.chan@mongodb.com', 'username': 'jasonjhchan'}Message: |