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

PYTHON-539 Timeouts in ResponseFuture should notify the conviction policy #1127

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

absurdfarce
Copy link
Collaborator

No description provided.

@absurdfarce absurdfarce self-assigned this Jun 8, 2022
…when

calling Cluster.signal_connection_failure (which we do in the unit test).
@absurdfarce
Copy link
Collaborator Author

AppVeyor failure is from a few random test failures, which (unfortunately) happens from time to time with those builds. I believe the Jenkins failures stem from an issue with the Jenkins server in question; this needs some more work.

@absurdfarce
Copy link
Collaborator Author

After looking into this today I've determined that the Jenkins failures do represent legit breakage with this PR. Can't tell yet if it's an unintended consequence of these changes or if the tests are making too many assumptions... still working on sorting that out.

@absurdfarce
Copy link
Collaborator Author

Test failures are also occurring on master; we've definitely got an issue to sort out there. I've filed PYTHON-1304 to address those issues (and possibly to get us back to a clean build on Jenkins generally).

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.

1 participant