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

YQ-4133 fix race on RM initialization #14931

Conversation

GrigoriyPA
Copy link
Collaborator

@GrigoriyPA GrigoriyPA commented Feb 23, 2025

Changelog entry

Fixed race on RM initialization, issue: #14932

Changelog category

  • Bugfix

Description for reviewers

@GrigoriyPA GrigoriyPA requested a review from a team as a code owner February 23, 2025 10:50
Copy link

github-actions bot commented Feb 23, 2025

🟢 2025-02-23 10:57:10 UTC The validation of the Pull Request description is successful.

Copy link

github-actions bot commented Feb 23, 2025

2025-02-23 10:54:46 UTC Pre-commit check linux-x86_64-relwithdebinfo for 79e8eee has started.
2025-02-23 10:56:41 UTC Artifacts will be uploaded here
2025-02-23 10:59:31 UTC ya make is running...
🟡 2025-02-23 11:55:55 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
18841 17488 0 2 1220 131

2025-02-23 11:57:41 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-02-23 12:20:04 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
203 (only retried tests) 74 0 0 1 128

🟢 2025-02-23 12:20:11 UTC Build successful.
🟡 2025-02-23 12:20:31 UTC ydbd size 2.1 GiB changed* by +1.1 MiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 1041280 merge: 79e8eee diff diff %
ydbd size 2 241 693 936 Bytes 2 242 819 824 Bytes +1.1 MiB +0.050%
ydbd stripped size 474 795 416 Bytes 474 854 072 Bytes +57.3 KiB +0.012%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Feb 23, 2025

2025-02-23 10:54:59 UTC Pre-commit check linux-x86_64-release-asan for 79e8eee has started.
2025-02-23 10:56:40 UTC Artifacts will be uploaded here
2025-02-23 10:59:28 UTC ya make is running...
🟡 2025-02-23 12:00:09 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11739 11641 0 49 14 35

2025-02-23 12:01:15 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-02-23 12:18:58 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
149 (only retried tests) 106 0 0 11 32

🟢 2025-02-23 12:19:06 UTC Build successful.
🟢 2025-02-23 12:19:35 UTC ydbd size 3.6 GiB changed* by -128.9 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 1041280 merge: 79e8eee diff diff %
ydbd size 3 899 596 224 Bytes 3 899 464 248 Bytes -128.9 KiB -0.003%
ydbd stripped size 1 364 895 696 Bytes 1 365 151 312 Bytes +249.6 KiB +0.019%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@GrigoriyPA GrigoriyPA merged commit 388cf79 into ydb-platform:main Feb 24, 2025
17 of 18 checks passed
@GrigoriyPA GrigoriyPA deleted the YQ-4133-fix-kqprun-fault-with-large-system-pool branch February 24, 2025 07:41
lberserq pushed a commit to lberserq/ydb that referenced this pull request Mar 3, 2025
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.

2 participants