Skip to content

Conversation

kevinAlbs
Copy link
Collaborator

@kevinAlbs kevinAlbs commented Sep 5, 2025

Follow-up to #2095 intended to fix observed test failures on macOS replica sets. Example:

test error in: /Users/ec2-user/data/mci/c4f6b76328d456c45e1a0f91c797f34d/mongoc/src/libmongoc/tests/test-mongoc-exhaust.c 202:test_exhaust_cursor()
Got unexpected total stream count to localhost:27018:
    Expected 3, got 4

I reproduced the error by running test in repeat 100x: https://spruce.mongodb.com/version/68bb4bca9fc72c00079c4923
Tests pass with this fix applied: https://spruce.mongodb.com/version/68bb4f4fe5543500071df4fc

I expect the extra observed connection is due to the monitoring connections taking time to establish. This fix awaits monitoring connections before proceeding with the test.

@kevinAlbs kevinAlbs marked this pull request as ready for review September 5, 2025 21:06
@kevinAlbs kevinAlbs requested a review from a team as a code owner September 5, 2025 21:06
Copy link
Contributor

@rcsanchez97 rcsanchez97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kevinAlbs kevinAlbs merged commit af71688 into mongodb:master Sep 8, 2025
44 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants