Fix driver.wait hang

[UgnineSirdis]

Retry reestablishing connection during driver start

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Fix driver.wait hang. If driver gets error during connection, it hangs waiting for connection, but the connection does not retry to reestablish. It reproduces in real tests.

Other information

[Copilot]

Pull request overview

This PR targets a startup reliability issue where (with discovery disabled) an initial connection failure could leave the driver waiting indefinitely for an endpoint that will never appear. It introduces retry/cleanup behavior (notably in the async driver path) and tightens connection teardown to avoid retaining stale channel/stub state.

Changes:

• Add retry loop for initial connection establishment when discovery is disabled in the async connection pool, with cancellation-safe cleanup.
• Ensure connection cache cleanup on stop() when discovery is disabled (sync + async paths).
• Make connection destroy() clear cached stubs and null out the channel; add tests for cleanup and async retry behavior.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
ydb/pool.py	Ensure cached connections are cleaned up on stop when discovery is disabled (no discovery thread).
ydb/connection.py	Harden teardown by clearing stub cache and dropping the channel reference during destroy.
ydb/aio/pool.py	Retry initial connection when discovery is disabled; add stopping flag and ensure cleanup on stop/cancel.
ydb/aio/connection.py	Mirror sync destroy improvements (clear stubs, drop channel reference).
tests/test_disable_discovery.py	Add regression tests for sync stop cleanup and async initial connection retry.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

[github-actions]

🌋 SLO Test Results

🟢 2 workload(s) tested — All thresholds passed

Commit: da25edd · View run

Workload	Thresholds	Duration	Report
sync-query	🟢 OK	10m 11s	📄 Report
sync-table	🟢 OK	10m 3s	📄 Report

---

Generated by ydb-slo-action