Skip to content

Add UT for group cache, improve SentToWarden for local messages #22541

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

Merged
merged 3 commits into from
Aug 8, 2025

Conversation

serbel324
Copy link
Collaborator

@serbel324 serbel324 commented Aug 7, 2025

Changelog entry

Add unit tests for storage group configuration propagation via DistConf cache. Improve BSC's SendToWarden method, send message directly to local warden.

Changelog category

  • Improvement

Copy link

github-actions bot commented Aug 7, 2025

🟢 2025-08-07 17:35:06 UTC The validation of the Pull Request description is successful.

Copy link

github-actions bot commented Aug 7, 2025

2025-08-07 17:36:24 UTC Pre-commit check linux-x86_64-release-asan for ea0a50e has started.
2025-08-07 17:36:40 UTC Artifacts will be uploaded here
2025-08-07 17:37:21 UTC Check cancelled

Copy link

github-actions bot commented Aug 7, 2025

2025-08-07 17:36:43 UTC Pre-commit check linux-x86_64-relwithdebinfo for ea0a50e has started.
2025-08-07 17:36:58 UTC Artifacts will be uploaded here
2025-08-07 17:37:21 UTC Check cancelled

Copy link

github-actions bot commented Aug 7, 2025

2025-08-07 17:39:48 UTC Pre-commit check linux-x86_64-release-asan for e40725f has started.
2025-08-07 17:40:03 UTC Artifacts will be uploaded here
2025-08-07 17:43:22 UTC ya make is running...
🟡 2025-08-07 19:31:36 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14744 14323 0 96 302 23

🟢 2025-08-07 19:32:59 UTC Build successful.
🟡 2025-08-07 19:33:25 UTC ydbd size 4.0 GiB changed* by +126.7 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: f1c78aa merge: e40725f diff diff %
ydbd size 4 247 870 800 Bytes 4 248 000 544 Bytes +126.7 KiB +0.003%
ydbd stripped size 1 469 950 552 Bytes 1 469 950 552 Bytes 0 Bytes 0.000%

*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 Aug 7, 2025

2025-08-07 17:39:50 UTC Pre-commit check linux-x86_64-relwithdebinfo for e40725f has started.
2025-08-07 17:40:05 UTC Artifacts will be uploaded here
2025-08-07 17:43:27 UTC ya make is running...
🟡 2025-08-07 19:00:42 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?
37278 34487 0 4 2752 35

2025-08-07 19:04:02 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-08-07 19:18:15 UTC Some tests failed, follow the links below. Going to retry failed tests...

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1010 (only retried tests) 934 0 1 53 22

2025-08-07 19:18:29 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-08-07 19:31:45 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
664 (only retried tests) 593 0 0 50 21

🟢 2025-08-07 19:31:56 UTC Build successful.
🟢 2025-08-07 19:32:17 UTC ydbd size 2.2 GiB changed* by +160 Bytes, which is < 100.0 KiB vs main: OK

ydbd size dash main: f1c78aa merge: e40725f diff diff %
ydbd size 2 414 561 096 Bytes 2 414 561 256 Bytes +160 Bytes +0.000%
ydbd stripped size 504 906 152 Bytes 504 906 216 Bytes +64 Bytes +0.000%

*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 Aug 8, 2025

2025-08-08 08:53:12 UTC Pre-commit check linux-x86_64-release-asan for 732f60a has started.
2025-08-08 08:53:26 UTC Artifacts will be uploaded here
2025-08-08 08:56:53 UTC ya make is running...
🟡 2025-08-08 10:49:19 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14981 14552 0 100 304 25

🟢 2025-08-08 10:50:40 UTC Build successful.
🟢 2025-08-08 10:51:07 UTC ydbd size 4.0 GiB changed* by -3.9 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: b51b4d0 merge: 732f60a diff diff %
ydbd size 4 248 371 632 Bytes 4 248 367 600 Bytes -3.9 KiB -0.000%
ydbd stripped size 1 470 079 064 Bytes 1 470 079 320 Bytes +256 Bytes +0.000%

*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 Aug 8, 2025

2025-08-08 08:53:18 UTC Pre-commit check linux-x86_64-relwithdebinfo for 732f60a has started.
2025-08-08 08:53:31 UTC Artifacts will be uploaded here
2025-08-08 08:56:49 UTC ya make is running...
🟡 2025-08-08 10:13:38 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?
37519 34728 0 6 2752 33

2025-08-08 10:17:26 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-08-08 10:30:08 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
860 (only retried tests) 786 0 0 53 21

🟢 2025-08-08 10:30:18 UTC Build successful.
🟢 2025-08-08 10:30:38 UTC ydbd size 2.2 GiB changed* by -4.0 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 6966047 merge: 732f60a diff diff %
ydbd size 2 414 811 176 Bytes 2 414 807 112 Bytes -4.0 KiB -0.000%
ydbd stripped size 504 942 632 Bytes 504 942 632 Bytes 0 Bytes 0.000%

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

@serbel324 serbel324 merged commit f2c19c8 into ydb-platform:main Aug 8, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants