[SERVER-31450] Add comment/warning to check_uuids_consistent_across_cluster.js that the JS test must have live primaries Created: 06/Oct/17 Updated: 30/Oct/23 Resolved: 03/Nov/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 3.6.0-rc3 |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | Dianna Hohensee (Inactive) | Assignee: | Dianna Hohensee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Sprint: | Sharding 2017-11-13 | ||||
| Participants: | |||||
| Linked BF Score: | 0 | ||||
| Description |
|
check_uuids_consistent_across_cluster.js (ShardingTest:: checkUUIDsConsistentAcrossCluster) can only be used on JS tests that have shard and config primaries available by the end of the test. Otherwise the aggregation query can fail, as in BF-6748. This should be clearly documented in check_uuids_consistent_across_cluster.js so it can be better used and so that it will be easier to diagnose the inevitable JS tests we have failed, or will fail, to prevent the check from running upon. |
| Comments |
| Comment by Githook User [ 03/Nov/17 ] |
|
Author: {'name': 'Dianna Hohensee', 'username': 'DiannaHohensee', 'email': 'dianna.hohensee@10gen.com'}Message: |
| Comment by Dianna Hohensee (Inactive) [ 10/Oct/17 ] |
|
I meant this as a file user comment, proclaiming when the check should be prevented from running on a js test – as well as how to disable it. |
| Comment by Esha Maharishi (Inactive) [ 10/Oct/17 ] |
|
Adding comment as part of |