[SERVER-21994] cleanup_orphaned_basic.js Created: 18/Dec/15 Updated: 18/Nov/16 Resolved: 28/Jan/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | 3.2.0 |
| Fix Version/s: | 3.2.5, 3.3.2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Adam Midvidy | Assignee: | Misha Tyulenev |
| Resolution: | Done | Votes: | 0 |
| Labels: | test-only | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Operating System: | ALL | ||||
| Backport Completed: | |||||
| Participants: | |||||
| Linked BF Score: | 0 | ||||
| Description |
sharding failed on linux-64-duroffcleanup_orphaned_basic.js - Logs | History BF Ticket Generated by adam.midvidy |
| Comments |
| Comment by Githook User [ 10/Mar/16 ] | ||||||||||||||
|
Author: {u'username': u'mikety', u'name': u'Misha Tyulenev', u'email': u'misha@mongodb.com'}Message: (cherry picked from commit df8e729bf9866081b670a4622140756542e09cc1) | ||||||||||||||
| Comment by Misha Tyulenev [ 04/Jan/16 ] | ||||||||||||||
|
ian.whalen seems that the issue is different: This fails at the unsharded collection not a bad namespace as the original BF ticket. assert.commandFailed(shardAdmin.runCommand( {cleanupOrphaned: ns}));
| ||||||||||||||
| Comment by Ian Whalen (Inactive) [ 04/Jan/16 ] | ||||||||||||||
|
misha.tyulenev we just saw a similar failure happen again on master here: sharding on solaris Can you please investigate if this problem is ongoing? if it's actually a different problem, just let me know and I'll open a new issue to track. | ||||||||||||||
| Comment by Misha Tyulenev [ 22/Dec/15 ] | ||||||||||||||
|
Need to backport to 3.2 | ||||||||||||||
| Comment by Githook User [ 21/Dec/15 ] | ||||||||||||||
|
Author: {u'username': u'mikety', u'name': u'Misha Tyulenev', u'email': u'misha@mongodb.com'}Message: | ||||||||||||||
| Comment by Misha Tyulenev [ 18/Dec/15 ] | ||||||||||||||
|
Agreed | ||||||||||||||
| Comment by Kaloian Manassiev [ 18/Dec/15 ] | ||||||||||||||
|
Misha, I had looked at this a little bit before and I think the problem is that the command itself doesn't check for validity of the namespace, but we rely on it happening later in the call chain:
I think the fix would be to first find out where does the check for invalid namespace happen in the cases where the test succeeds and then to add a check within the cleanupOrphaned command itself. | ||||||||||||||
| Comment by Daniel Pasette (Inactive) [ 18/Dec/15 ] | ||||||||||||||
|
I think misha recently added this test, which may make it easier for him to diagnose. | ||||||||||||||
| Comment by Adam Midvidy [ 18/Dec/15 ] | ||||||||||||||
| ||||||||||||||
| Comment by Adam Midvidy [ 18/Dec/15 ] | ||||||||||||||
|
kaloian.manassiev can you please triage? |