Skip to content

fix(py-drv): use fixed python driver with proper reconnections#14708

Closed
vponomaryov wants to merge 2 commits into
scylladb:masterfrom
vponomaryov:fix-pydrv-race-issue-614
Closed

fix(py-drv): use fixed python driver with proper reconnections#14708
vponomaryov wants to merge 2 commits into
scylladb:masterfrom
vponomaryov:fix-pydrv-race-issue-614

Conversation

@vponomaryov

@vponomaryov vponomaryov commented May 20, 2026

Copy link
Copy Markdown
Contributor

Testing

  • [ ]

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Reminders

  • Add New configuration option and document them (in sdcm/sct_config.py)
  • Add unit tests to cover my changes (under unit-test/ folder)
  • Update the Readme/doc folder relevant to this change (if needed)

@vponomaryov vponomaryov added the New Hydra Version PR# introduces new Hydra version label May 20, 2026
@vponomaryov vponomaryov marked this pull request as draft May 20, 2026 10:47
@scylladb-promoter

scylladb-promoter commented May 20, 2026

Copy link
Copy Markdown
Collaborator

✅ Test Summary: PASSED

✅ Precommit: PASSED

Total Passed Failed Skipped
38 16 0 22

✅ Tests: PASSED

Total Passed Failed Errors Skipped
3217 3194 0 0 23

Full build log

@vponomaryov vponomaryov force-pushed the fix-pydrv-race-issue-614 branch from d173d87 to 34dbe41 Compare May 21, 2026 10:25
@vponomaryov vponomaryov added backport/none Backport is not required New Hydra Version PR# introduces new Hydra version and removed New Hydra Version PR# introduces new Hydra version labels May 21, 2026
@vponomaryov

Copy link
Copy Markdown
Contributor Author

The fix worked just great.
See comment: scylladb/python-driver#889 (comment)

@vponomaryov

Copy link
Copy Markdown
Contributor Author

The fix for the py-drv bug was merged:

So, close this PR and create a new one when new driver version gets released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/none Backport is not required New Hydra Version PR# introduces new Hydra version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants