Skip to content

test(NODE-6802): unskip operations against non-load balanced clusters fail if URI contains loadBalanced=true on latest #4484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 25, 2025
Merged

test(NODE-6802): unskip operations against non-load balanced clusters fail if URI contains loadBalanced=true on latest #4484

merged 1 commit into from
Mar 25, 2025

Conversation

nbbeeken
Copy link
Contributor

@nbbeeken nbbeeken commented Mar 24, 2025 •
edited
Loading

Description

What is changing?

Unskip lb test on latest

spec: mongodb/specifications@f04df7d

NODE-6802

update: This test runs on non-lb hosts
https://parsley.mongodb.com/evergreen/mongo_node_driver_next_rhel8_no_auth_tests_test_latest_server_noauth_patch_5ce0a4ec855fd1d609fbb17f85b014e5b0fa8157_67e1c3fbf504270007ffbd67_25_03_24_20_43_41/0/task?bookmarks=0,11408&shareLine=7718

Is there new documentation needed for these changes?

No

What is the motivation for this change?

spec compat

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@nbbeeken
test(NODE-6802): unskip operations against non-load balanced clusters…
c27bbd5 
… fail if URI contains loadBalanced=true on latest
@nbbeeken nbbeeken marked this pull request as ready for review March 24, 2025 21:10
@nbbeeken nbbeeken requested a review from a team as a code owner March 24, 2025 21:10
@baileympearson baileympearson self-assigned this Mar 25, 2025
@baileympearson baileympearson added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Mar 25, 2025
@baileympearson baileympearson merged commit d8caa42 into main Mar 25, 2025
26 of 30 checks passed
@baileympearson baileympearson deleted the NODE-6802-lb-conn-estab branch March 25, 2025 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baileympearson baileympearson baileympearson approved these changes

Assignees

@baileympearson baileympearson

Labels
Primary Review In Review with primary reviewer, not yet ready for team's eyes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nbbeeken @baileympearson