Skip to content

Correct temporary PDisk start sequence #7156

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

Conversation

alexvru
Copy link
Collaborator

@alexvru alexvru commented Jul 26, 2024

Changelog entry

Correct temporary PDisk start sequence

Changelog category

  • Bugfix

Additional information

Temporary PDisks now spin up their VDisks correctly.

Copy link

github-actions bot commented Jul 26, 2024

2024-07-26 17:10:27 UTC Pre-commit check for 16e6a47 has started.
2024-07-26 17:13:27 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-07-26 17:19:31 UTC Build successful.

Copy link

github-actions bot commented Jul 26, 2024

2024-07-26 17:12:20 UTC Pre-commit check for 16e6a47 has started.
2024-07-26 17:15:10 UTC Check linux-x86_64-release-asan is running...
🔴 2024-07-26 19:25:50 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9897 9308 0 85 328 176

🟢 2024-07-26 19:26:48 UTC Build successful.
🟡 2024-07-26 19:27:46 UTC ydbd size 5.2 GiB changed* by +207.1 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 9785f6a merge: 16e6a47 diff diff %
ydbd size 5 565 674 232 Bytes 5 565 886 280 Bytes +207.1 KiB +0.004%
ydbd stripped size 1 198 426 800 Bytes 1 198 472 624 Bytes +44.8 KiB +0.004%

*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 Jul 26, 2024

2024-07-26 17:12:47 UTC Pre-commit check for 16e6a47 has started.
2024-07-26 17:15:42 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-07-26 18:47:14 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14184 12846 0 1 1325 12

2024-07-26 18:48:53 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-07-26 18:56:19 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
12 (only retried tests) 4 0 0 0 8

🟢 2024-07-26 18:57:29 UTC Build successful.
🟡 2024-07-26 18:58:05 UTC ydbd size 8.1 GiB changed* by +620.0 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 9785f6a merge: 16e6a47 diff diff %
ydbd size 8 668 833 544 Bytes 8 669 468 440 Bytes +620.0 KiB +0.007%
ydbd stripped size 471 575 616 Bytes 471 597 824 Bytes +21.7 KiB +0.005%

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

@alexvru alexvru merged commit 4dbd617 into ydb-platform:main Jul 26, 2024
11 of 13 checks passed
@alexvru alexvru deleted the nodewarden/correct-temporary-pdisk-start branch July 26, 2024 21:06
MrLolthe1st pushed a commit to MrLolthe1st/ydb that referenced this pull request Aug 12, 2024
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