[SERVER-11035] count10.js failing on some Enterprise MCI builders Created: 04/Oct/13 Updated: 11/Jul/16 Resolved: 04/Oct/13 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Testing Infrastructure |
| Affects Version/s: | None |
| Fix Version/s: | 2.5.3 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Matt Kangas | Assignee: | David Storch |
| Resolution: | Done | Votes: | 0 |
| Labels: | buildbot | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Operating System: | ALL | ||||||||
| Participants: | |||||||||
| Description |
|
David - I merged your pull request 511 for Code review: http://codereview.10gen.com/11598025/, reviewer was scotthernandez — As of now (11 PM Sept 3) , I see two builders that have been failing on your new count10.js test since that commit: enterprise-suse11-64: http://mci.10gen.com/ui/task/mongodb_mongo_master_enterprise_suse11_64_1819d0083db9d71560a886598a245f12704d4403_13_10_03_22_34_12_js_auth_suse11 enterprise-ubuntu1204-64: http://mci.10gen.com/ui/task/mongodb_mongo_master_enterprise_ubuntu1204_64_1819d0083db9d71560a886598a245f12704d4403_13_10_03_22_34_12_js_auth_ubuntu1204
|
| Comments |
| Comment by auto [ 04/Oct/13 ] | |||||||||||||||
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: Signed-off-by: Ian Whalen <ian@mongodb.com> | |||||||||||||||
| Comment by Ian Whalen (Inactive) [ 04/Oct/13 ] | |||||||||||||||
|
just saw it pass on js_auth here: http://mci.10gen.com/ui/task/mongodb_mongo_master_enterprise_suse11_64_37a7f2f07fddce4e413a56753473a7333da35432_13_10_04_19_01_34_js_auth_suse11 applying patch now. | |||||||||||||||
| Comment by Scott Hernandez (Inactive) [ 04/Oct/13 ] | |||||||||||||||
|
Matt, let's just commit this since David ran it locally on the failing tests and was able to show it worked with spencer help. We can then cancel/delete the mci patch. | |||||||||||||||
| Comment by David Storch [ 04/Oct/13 ] | |||||||||||||||
|
Rewrote the test to use startParallelShell instead so that auth works correctly, and spencer reviewed. MCI patch: http://mci.10gen.com/ui/patch/524f108d3ff1221bb8000002. If things pass this time, the matching pull request is here: https://github.com/mongodb/mongo/pull/514. | |||||||||||||||
| Comment by Scott Hernandez (Inactive) [ 04/Oct/13 ] | |||||||||||||||
|
spencer was helping and last I saw a version in review which should auth in the async callback that should work, but I haven't see the commit/mci-patch run yet. The issue was that the async thread wasn't auth'd. | |||||||||||||||
| Comment by Matt Kangas [ 04/Oct/13 ] | |||||||||||||||
|
David, your second MCI patch still seems to have failed dur_passthrough.js on Linux 64 due to count10.js http://mci.10gen.com/ui/patch/524ed7ea3ff12216b7000003
Did you submit a later patch that ran cleanly? I am out of the office today; please grab Scott to help you if you're not sure how to resolve this. (since he reviewed your request originally) | |||||||||||||||
| Comment by David Storch [ 04/Oct/13 ] | |||||||||||||||
|
Code review at this pull request: https://github.com/mongodb/mongo/pull/513 | |||||||||||||||
| Comment by David Storch [ 04/Oct/13 ] | |||||||||||||||
|
After more investigation, the problem is auth related, not timing related. New MCI patch: http://mci.10gen.com/ui/patch/524ed7ea3ff12216b7000003. This should do the trick. | |||||||||||||||
| Comment by David Storch [ 04/Oct/13 ] | |||||||||||||||
|
Hopefully this is just a timing issue when things are running slower on MCI. I pushed a patch to MCI to test: http://mci.10gen.com/ui/patch/524ec7083ff1224029000005 | |||||||||||||||
| Comment by Eric Milkie [ 04/Oct/13 ] | |||||||||||||||
|
Buildbot plain jane Linux 64 is failing in dur_passthrough.js due to the same problem. It's not Enterprise specific. | |||||||||||||||
| Comment by Matt Kangas [ 04/Oct/13 ] | |||||||||||||||
|
Other "passthrough" tests that failed on count10.js also:
|