Skip to content

Conversation

@LebedevOleg1
Copy link
Collaborator

Changelog entry

...

Changelog category

  • Not for changelog (changelog entry is not required)

Description for reviewers

...

@github-actions
Copy link

🟢 2025-10-27 13:50:19 UTC The validation of the Pull Request description is successful.

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

2025-10-27 13:51:17 UTC Pre-commit check linux-x86_64-relwithdebinfo for f4c4dec has started.
2025-10-27 13:51:31 UTC Artifacts will be uploaded here
2025-10-27 13:52:54 UTC ya make is running...
🟢 2025-10-27 13:56:14 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
31 31 0 0 0 0

🟢 2025-10-27 13:56:16 UTC Build successful.
🔴 2025-10-27 13:56:27 UTC ydbd size 2.3 GiB changed* by +2.8 MiB, which is >= 2.0 MiB vs main: Alert

ydbd size dash main: aa1cb3a merge: f4c4dec diff diff %
ydbd size 2 437 018 600 Bytes 2 439 938 760 Bytes +2.8 MiB +0.120%
ydbd stripped size 518 366 664 Bytes 518 917 576 Bytes +538.0 KiB +0.106%

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

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

2025-10-27 13:51:18 UTC Pre-commit check linux-x86_64-release-asan for f4c4dec has started.
2025-10-27 13:51:34 UTC Artifacts will be uploaded here
2025-10-27 13:52:51 UTC ya make is running...
🟢 2025-10-27 13:54:04 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3 3 0 0 0 0

🟢 2025-10-27 13:54:07 UTC Build successful.
🔴 2025-10-27 13:54:21 UTC ydbd size 3.8 GiB changed* by +4.4 MiB, which is >= 2.0 MiB vs main: Alert

ydbd size dash main: aa1cb3a merge: f4c4dec diff diff %
ydbd size 4 055 911 368 Bytes 4 060 520 320 Bytes +4.4 MiB +0.114%
ydbd stripped size 1 506 943 104 Bytes 1 508 592 960 Bytes +1.6 MiB +0.109%

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

@Enjection
Copy link
Member

Logically looks fine, but as I said earlier, framework became essential. If test can't fit even 1.5 screens it is bad, nobody will ever debug it when it become flaky or broken.

@github-actions
Copy link

github-actions bot commented Oct 28, 2025

2025-10-28 07:31:54 UTC Pre-commit check linux-x86_64-relwithdebinfo for 2b80ae2 has started.
2025-10-28 07:32:10 UTC Artifacts will be uploaded here
2025-10-28 07:33:30 UTC ya make is running...
🟢 2025-10-28 07:35:52 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
31 31 0 0 0 0

🟢 2025-10-28 07:35:55 UTC Build successful.
🟢 2025-10-28 07:36:05 UTC ydbd size 2.3 GiB changed* by 0 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 43c912e merge: 2b80ae2 diff diff %
ydbd size 2 427 164 280 Bytes 2 427 164 280 Bytes 0 Bytes 0.000%
ydbd stripped size 517 538 536 Bytes 517 538 536 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

@github-actions
Copy link

github-actions bot commented Oct 28, 2025

2025-10-28 07:32:19 UTC Pre-commit check linux-x86_64-release-asan for 2b80ae2 has started.
2025-10-28 07:32:36 UTC Artifacts will be uploaded here
2025-10-28 07:33:55 UTC ya make is running...
🟢 2025-10-28 07:35:12 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3 3 0 0 0 0

🟢 2025-10-28 07:35:14 UTC Build successful.
🟢 2025-10-28 07:35:28 UTC ydbd size 3.8 GiB changed* by -96 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 43c912e merge: 2b80ae2 diff diff %
ydbd size 4 062 498 864 Bytes 4 062 498 768 Bytes -96 Bytes -0.000%
ydbd stripped size 1 509 406 328 Bytes 1 509 406 264 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

@LebedevOleg1 LebedevOleg1 self-assigned this Oct 28, 2025
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