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

Fix connection error handling in database resolver #2990

Merged
merged 2 commits into from
Mar 20, 2024

Conversation

vitalyisaev2
Copy link
Member

Changelog entry

  • Fix connection error handling in database resolver

Changelog category

  • Bugfix

Additional information

YDB crashed due to connection timeout while connecting to YDB CP endpoint.

@vitalyisaev2 vitalyisaev2 requested review from a team as code owners March 20, 2024 10:10
Copy link

github-actions bot commented Mar 20, 2024

2024-03-20 10:18:43 UTC Pre-commit check for b334e14 has started.
2024-03-20 10:18:45 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-20 10:34:35 UTC Build successful.

Copy link

github-actions bot commented Mar 20, 2024

2024-03-20 13:54:21 UTC Pre-commit check for b334e14 has started.
2024-03-20 13:54:22 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-20 13:55:21 UTC Build successful.
2024-03-20 13:55:30 UTC Tests are running...
🔴 2024-03-20 14:22:23 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15109 14490 0 81 507 31

Copy link

github-actions bot commented Mar 20, 2024

2024-03-20 13:49:48 UTC Pre-commit check for b334e14 has started.
2024-03-20 13:49:50 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-20 13:50:49 UTC Build successful.
2024-03-20 13:51:00 UTC Tests are running...
🔴 2024-03-20 14:31:09 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
68705 57671 0 11 10997 26

@vitalyisaev2 vitalyisaev2 merged commit acfab9b into ydb-platform:main Mar 20, 2024
3 of 5 checks passed
@shnikd shnikd mentioned this pull request Mar 26, 2024
This was referenced Mar 29, 2024
EgorkaZ pushed a commit to EgorkaZ/ydb that referenced this pull request Apr 5, 2024
EgorkaZ pushed a commit to EgorkaZ/ydb that referenced this pull request Apr 8, 2024
@shnikd shnikd mentioned this pull request Apr 11, 2024
vitalyisaev2 added a commit to vitalyisaev2/ydb that referenced this pull request May 21, 2024
vitalyisaev2 added a commit that referenced this pull request May 21, 2024
* Fix connection error handling in database resolver (#2990)
* Always add 'u-' prefix to the dedicated YDB database endpoint during resolving (#4688)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants