Skip to content
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.

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

GODRIVER-2333 Assert that Ping op succeeds initial DNS spec tests #1124

Merged
merged 9 commits into from Nov 16, 2022

Conversation

prestonvasquez
Copy link
Collaborator

GODRIVER-2333

Summary

Sync mongo/specifications b2e35cb to

Run a "ping" operation unless ping is false or error is true.

Background & Motivation

This addition will potentially expose spec test misconfigurations by running an operation on the server defined by the test connection string, rather than simply listing hosts.

@prestonvasquez prestonvasquez requested review from benjirewis and matthewdale and removed request for matthewdale and benjirewis November 8, 2022 02:39
@prestonvasquez prestonvasquez requested review from matthewdale and removed request for qingyang-hu November 11, 2022 17:48
Copy link
Contributor

@benjirewis benjirewis left a comment

Choose a reason for hiding this comment

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

Looking good!

Copy link
Contributor

@benjirewis benjirewis left a comment

Choose a reason for hiding this comment

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

LGTM as long as tests pass! Thanks

Copy link
Collaborator

@matthewdale matthewdale left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@prestonvasquez prestonvasquez merged commit 2053ec7 into mongodb:master Nov 16, 2022
@prestonvasquez prestonvasquez deleted the GODRIVER-2333 branch November 16, 2022 20:11
prestonvasquez added a commit to prestonvasquez/mongo-go-driver that referenced this pull request Dec 8, 2022
prestonvasquez added a commit that referenced this pull request Dec 8, 2022
* GODRIVER-2651 Break NoWritesPerformed-Only Error Sequence (#1135)

Co-authored-by: Qingyang Hu <103950869+qingyang-hu@users.noreply.github.com>
Co-authored-by: Benjamin Rewis <32186188+benjirewis@users.noreply.github.com>
Co-authored-by: Kevin Albertson <kevin.albertson@mongodb.com>
Co-authored-by: Qingyang Hu <103950869+qingyang-hu@users.noreply.github.com>
Co-authored-by: Benjamin Rewis <32186188+benjirewis@users.noreply.github.com>

* GODRIVER-2333 Assert that Ping op succeeds  initial DNS spec tests (#1124)

* GODRIVER-2577 Retry heartbeat on timeout to prevent pool cleanup in FAAS pause. (#1133)

* resolve merge conflicts

Co-authored-by: Qingyang Hu <103950869+qingyang-hu@users.noreply.github.com>
Co-authored-by: Benjamin Rewis <32186188+benjirewis@users.noreply.github.com>
Co-authored-by: Kevin Albertson <kevin.albertson@mongodb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants