[CDRIVER-2279] /BulkOperation/OP_MSG/max_msg_size failure Created: 13/Sep/17  Updated: 28/Oct/23  Resolved: 13/Sep/17

Status: Closed
Project: C Driver
Component/s: libmongoc, tests
Affects Version/s: 1.9.0
Fix Version/s: 1.9.0

Type: Improvement Priority: Major - P3
Reporter: A. Jesse Jiryu Davis Assignee: Hannes Magnusson
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

/BulkOperation/OP_MSG/max_msg_size fails:

[2017/09/12 22:26:48.930] + make -o test-libmongoc test TEST_ARGS=--no-fork -d -F test-results.json
[2017/09/12 22:26:48.930] FAIL
[2017/09/12 22:26:48.930] Assert Failure: 2 == 1
[2017/09/12 22:26:48.930] tests/test-mongoc-bulk.c:3441  test_bulk_max_msg_size()

https://evergreen.mongodb.com/task/mongo_c_driver_clang38_i386_test_latest_sharded_cluster_noauth_nosasl_nossl_8f21723ca96c94aab5e131ec8c7ee94c9c501591_17_09_13_02_05_13

The assert is checking that ONE message was sent, but evidently two messages are sent instead.



 Comments   
Comment by Hannes Magnusson [ 13/Sep/17 ]

I don't think the travis failure is caused by this commit so closing.
There also is another ticket for the travis failure: CDRIVER-2281

Comment by Hannes Magnusson [ 13/Sep/17 ]

I got a failure mail from travis which I wasn't expecting and haven't yet looked at, but its failing something.

Comment by A. Jesse Jiryu Davis [ 13/Sep/17 ]

Thanks for handling this Hannes. Can this issue be closed now?

Comment by Ramon Fernandez Marina [ 13/Sep/17 ]

Author:

{'username': u'bjori', 'name': u'Hannes Magnusson', 'email': u'bjori@php.net'}

Message:CDRIVER-2279 Subtract the size of the clusterTime
Branch:master
https://github.com/mongodb/mongo-c-driver/commit/154c7202d256ab3c6fff22becf1481c53a137cc1

Comment by Ramon Fernandez Marina [ 13/Sep/17 ]

Author:

{'username': u'bjori', 'name': u'Hannes Magnusson', 'email': u'bjori@php.net'}

Message:Revert "CDRIVER-2279 temporarily skip OP_MSG test"

This reverts commit 4d747ab42ed6fce195fc3d96cda534fc3299429a.
Branch:master
https://github.com/mongodb/mongo-c-driver/commit/fd3332df421a2258c1efcf0835f0c403f21924a1

Comment by Hannes Magnusson [ 13/Sep/17 ]

does the message now include $logicalTime or additional data that is appended differently?
This was passing just fine yesterday

Comment by Ramon Fernandez Marina [ 13/Sep/17 ]

Author:

{'username': u'ajdavis', 'name': u'A. Jesse Jiryu Davis', 'email': u'jesse@mongodb.com'}

Message:CDRIVER-2279 temporarily skip OP_MSG test
Branch:master
https://github.com/mongodb/mongo-c-driver/commit/4d747ab42ed6fce195fc3d96cda534fc3299429a

Generated at Wed Feb 07 21:14:44 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.