test(NODE-6771): sync non-lb-connection-establishment test#4426
Merged
durran merged 7 commits intomongodb:mainfrom Feb 24, 2025
Merged
test(NODE-6771): sync non-lb-connection-establishment test#4426durran merged 7 commits intomongodb:mainfrom
non-lb-connection-establishment test#4426durran merged 7 commits intomongodb:mainfrom
Conversation
Contributor
|
evergreen help |
Available Evergreen Comment Commands
|
Contributor
|
evergreen retry |
5 tasks
b7da57b to
57b3124
Compare
nbbeeken
requested changes
Feb 21, 2025
Contributor
nbbeeken
left a comment
There was a problem hiding this comment.
Did something in here break the coverage task? Seems new but could be an external change
Contributor
Looks unrelated but odd it's passing on main. The error is that the assumed role doesn't have permission to list the s3 bucket. I restarted it and it passed. |
durran
previously approved these changes
Feb 24, 2025
nbbeeken
requested changes
Feb 24, 2025
Contributor
@durran I agree, I just wanted to double-check check we looked into it to be sure, ty! |
nbbeeken
approved these changes
Feb 24, 2025
5 tasks
durran
approved these changes
Feb 24, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
What is changing?
Sync
non-lb-connection-establishmenttest to mongodb/specifications@d05c33eIs there new documentation needed for these changes?
No.
What is the motivation for this change?
Intended to proactively avoid tests failure once drivers start testing 8.1 builds. See DRIVERS-3108. Drivers are not-yet testing 8.1 (see slack).
Release Highlight
Fill in title or leave empty for no highlight
Double check the following
[ ] Rannpm run check:lintscript[ ] Self-review completed using the steps outlined heretype(NODE-xxxx)[!]: descriptionfeat(NODE-1234)!: rewriting everything in coffeescript[ ] Changes are covered by tests[ ] New TODOs have a related JIRA ticket