[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: PNG File rc3CXXerror.PNG     HTML File sconsOutput    

 Description   

Hi,
I've updated mongo server to rc3. However, when I run the tests for cxx driver, I'm getting the failure in unit tests as shown in the snapshot attached.

I also tried testing on Ubuntu PPC but that also gives the same results.

Kindly guide me on this issue.

Regards,
valnci

System Info:
OS/Arch - RHELppc and UbuntuPPC
MongoDB shell version: 3.2.0-rc3-24-g2508a0f
Mongo Orchestration v0.6.dev0



 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: CXX-744 Fix incorrect types used to hold 64 bit values
Branch: legacy
https://github.com/mongodb/mongo-cxx-driver/commit/69f2088dfec6107d0b62eee2a9cbebad3120c773

Comment by Andrew Morrow (Inactive) [ 30/Nov/15 ]

https://github.com/mongodb/mongo-cxx-driver/pull/379

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,
Valencia

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,
Valencia

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.

  • valnci
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)

Generated at Wed Feb 07 22:00:09 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.