[CXX-744] RandomTest.NextInt64SanityCheck Fails on PPC Created: 25/Nov/15 Updated: 07/Dec/15 Resolved: 30/Nov/15 |
|
| Status: | Closed |
| Project: | C++ Driver |
| Component/s: | Testing |
| Affects Version/s: | legacy-1.0.7 |
| Fix Version/s: | legacy-1.1.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Valencia Serrao | Assignee: | Andrew Morrow (Inactive) |
| Resolution: | Done | Votes: | 3 |
| Labels: | legacy-cxx | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Description |
|
Hi, I also tried testing on Ubuntu PPC but that also gives the same results. Kindly guide me on this issue. Regards, System Info: |
| Comments |
| Comment by Githook User [ 30/Nov/15 ] |
|
Author: {u'username': u'acmorrow', u'name': u'Andrew Morrow', u'email': u'acm@mongodb.com'}Message: |
| Comment by Andrew Morrow (Inactive) [ 30/Nov/15 ] |
| Comment by Valencia Serrao [ 30/Nov/15 ] |
|
Hi Andrew Yes, the fix has solved the failure. Modifying the test "NextInt64SanityCheck" as per the fix, solved the failure. I have also applied the additional fix for test "NextInt64InRange". Regards, |
| Comment by Valencia Serrao [ 30/Nov/15 ] |
|
Hi Andrew Thanks for the fix. I was facing a hardware issue with my machine, so could not try it. Now that the h/w issue is settled i'll report to you the results ASAP. Thanks, |
| Comment by Andrew Morrow (Inactive) [ 28/Nov/15 ] |
|
valnci where you able to test out the fix in my patch? It would be a big help to know if that resolves the issue for you. As to when to expect a fix for this, I'd expect a release containing this fix within the next two weeks. |
| Comment by Andrew Morrow (Inactive) [ 26/Nov/15 ] |
|
valnci - Before I can estimate when we will have a fix, I first need you to confirm for me whether you can fix the issue on your system by applying the patch I linked to above. You should just be able to make the same change by hand (it is only two lines). Here is the patch I would like you to test, again: https://github.com/acmorrow/mongo-cxx-driver/commit/be941ee1dff857c6ff49a48e5a323621aaa283c5 |
| Comment by Valencia Serrao [ 26/Nov/15 ] |
|
Hi Andrew, Thanks for the information. Do we expect a fix for this Failure in near future ? -valnci |
| Comment by Andrew Morrow (Inactive) [ 26/Nov/15 ] |
|
I believe this is a bug in the test introduced when backporting it from the server and changing C+11 to C+03. This should not affect the correctness of the driver. |
| Comment by Andrew Morrow (Inactive) [ 26/Nov/15 ] |
|
As I mentioned on mongodb-dev, this test doesn't interact with the server, so changing the server version won't make any difference. |
| Comment by Valencia Serrao [ 25/Nov/15 ] |
|
Hi, I've updated server to rc4. However, when I execute the unit tests the same Failure is observed.
|
| Comment by Valencia Serrao [ 25/Nov/15 ] |
|
PFA scons output of tests run on legacy-1.0.6 (named: sconsOutput) |
| Comment by Valencia Serrao [ 25/Nov/15 ] |
|
Information Update: As per earlier discussion with acm on mongodb-dev https://groups.google.com/forum/#!topic/mongodb-dev/BWcidH1EWaU, I've already tried running the tests on legacy-1.0.6, however, the RandonTest.NextInt64SanityCheck failure is not observed as the test is not there in source code. PFA the scons output for the test run on legacy-1.0.6 (name: sconsOutput) |