[SERVER-26626] In external_auth_WT JSTests, ensure that Saslauthd has fully started before continuing test Created: 13/Oct/16 Updated: 09/Oct/19 Resolved: 10/May/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 3.6.14, 4.1.12, 4.0.11 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Spencer Jackson | Assignee: | Jonathan Reams |
| Resolution: | Done | 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: | Security 2019-05-06, Security 2019-05-20 | ||||||||||||
| Participants: | |||||||||||||
| Linked BF Score: | 50 | ||||||||||||
| Description |
|
In the test framework, because we're managing the lifecycle of external processes we're dependent on, it seems like it is possible to start saslauthd, but for saslauthd to not have fully started by the time MongoDB has started. To fix this, we should use runProgram, which will return once saslauthd has doubleforked, instead of _startMongoProgram which returns immediately. Then, we should repeatedly check for the existence of saslauthd's socket file. If the socket file exists, we have a good guarantee that saslauthd is running. |
| Comments |
| Comment by Githook User [ 07/Jun/19 ] |
|
Author: {'name': 'Jonathan Reams', 'email': 'jbreams@mongodb.com', 'username': 'jbreams'}Message: (cherry picked from commit a2310ac21157502b66dd034f034afc91d061e2a3) |
| Comment by Githook User [ 07/Jun/19 ] |
|
Author: {'name': 'Jonathan Reams', 'email': 'jbreams@mongodb.com', 'username': 'jbreams'}Message: (cherry picked from commit a2310ac21157502b66dd034f034afc91d061e2a3) |
| Comment by Githook User [ 10/May/19 ] |
|
Author: {'email': 'jbreams@mongodb.com', 'name': 'Jonathan Reams', 'username': 'jbreams'}Message: |