Skip to content

Conversation

Rattysed
Copy link
Collaborator

@Rattysed Rattysed commented Jun 19, 2025

#3614

Add GetClusterCapacity handler that collects cluster utilization information from YDB and DiskRegistry and returns it grouped by disk types.
This handler will be used by Disk Manager to intelligently distribute newly created disks across shards.

For collecting data from Disk Registry, an analogous handler has been added. For collecting from YDB, GetStorageStatsRequest is used (https://github.com/ydb-platform/ydb/blob/911050b753e7fb532f13324f65d2a167e8045e37/ydb/core/protos/sys_view.proto#L353-L358)

@SvartMetal SvartMetal self-requested a review June 19, 2025 16:56
Copy link
Contributor

github-actions bot commented Jun 20, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit cc7d416.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8666 8663 0 1 1 1

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit cc7d416.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
45 44 0 0 0 1

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit cc7d416.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
43 42 0 0 0 1

drbasic
drbasic previously approved these changes Jun 23, 2025
Copy link
Contributor

github-actions bot commented Jun 30, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 666caee.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8723 8707 0 0 8 8

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 666caee.

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

Copy link
Contributor

github-actions bot commented Jun 30, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 851ed53.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8690 8677 0 3 10 0

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 851ed53.

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

@Rattysed Rattysed marked this pull request as draft July 1, 2025 11:50
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 6e8457a.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8691 8664 0 5 22 0

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 6e8457a.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
342 328 0 2 12 0

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 6e8457a.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
329 315 0 2 12 0

Copy link
Contributor

github-actions bot commented Jul 4, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit adffe69.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8691 8675 0 3 13 0

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit adffe69.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
329 315 0 2 12 0

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit adffe69.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
329 315 0 2 12 0

@Rattysed Rattysed marked this pull request as ready for review July 21, 2025 08:39
@Rattysed Rattysed force-pushed the users/rattysed/issue-3614-getclustercapacity-handler branch from adffe69 to 198d1b2 Compare July 21, 2025 10:06
@Rattysed Rattysed added blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR labels Jul 21, 2025
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit c0cd413.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
5526 5525 0 0 0 1 0

@Rattysed Rattysed requested review from SvartMetal and drbasic July 23, 2025 15:12
SvartMetal
SvartMetal previously approved these changes Jul 23, 2025
drbasic
drbasic previously approved these changes Jul 23, 2025
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit dd6a9a9.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
5526 5525 0 0 0 1 0

@Rattysed Rattysed marked this pull request as draft July 24, 2025 22:49
@Rattysed
Copy link
Collaborator Author

ручка GetStorageStats не даёт информации о том, какой конкретно кластер YDB насколько занят. Внутри могут быть любые облачные сервисы, например NFS

@Rattysed Rattysed force-pushed the users/rattysed/issue-3614-getclustercapacity-handler branch from ed15a7b to 82dfa58 Compare August 4, 2025 20:30
Copy link
Contributor

github-actions bot commented Aug 4, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 82dfa58.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
5588 5587 0 0 0 1 0

@Rattysed Rattysed added asan Launch builds with address sanitizer along with regular build msan Launch builds with memory sanitizer along with regular build ubsan Launch builds with undefined behaviour sanitizer along with regular build labels Aug 6, 2025
Copy link
Contributor

github-actions bot commented Aug 6, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-release-msan: all tests PASSED for commit 82dfa58.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4936 4936 0 0 0 0 0

Copy link
Contributor

github-actions bot commented Aug 6, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-release-asan: all tests PASSED for commit 82dfa58.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4954 4954 0 0 0 0 0

Copy link
Contributor

github-actions bot commented Aug 6, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-release-ubsan: all tests PASSED for commit 82dfa58.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4967 4967 0 0 0 0 0

@Rattysed Rattysed merged commit dc263bc into main Aug 6, 2025
34 of 36 checks passed
@Rattysed Rattysed deleted the users/rattysed/issue-3614-getclustercapacity-handler branch August 6, 2025 10:36
Copy link
Contributor

github-actions bot commented Aug 6, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 82dfa58.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
5588 5587 0 0 0 1 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
asan Launch builds with address sanitizer along with regular build blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR msan Launch builds with memory sanitizer along with regular build ubsan Launch builds with undefined behaviour sanitizer along with regular build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants