Skip to content

issue-3371: skip reporting of ZeroBlocks metrics #3561

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

antonmyagkov
Copy link
Collaborator

@antonmyagkov antonmyagkov commented May 23, 2025

Skip reporting zeroblocks metrics as write metrics when the discard option is enabled, as this can mislead clients into overestimating the actual disk throughput. I added ReportZeroBlocksMetricsForYDBBasedDisks field to diagnostics config to control this behavior.

@antonmyagkov antonmyagkov added the blockstore Add this label to run only cloud/blockstore build and tests on PR label May 23, 2025
@antonmyagkov antonmyagkov force-pushed the users/myagkov/issue-3371-skip-zero-blocks-metrics branch from 442d460 to d33274f Compare May 23, 2025 13:57
@antonmyagkov antonmyagkov marked this pull request as ready for review May 23, 2025 15:35
@antonmyagkov antonmyagkov force-pushed the users/myagkov/issue-3371-skip-zero-blocks-metrics branch 2 times, most recently from ae0e8cf to db3fdfb Compare May 26, 2025 11:30
@antonmyagkov antonmyagkov force-pushed the users/myagkov/issue-3371-skip-zero-blocks-metrics branch from db3fdfb to 24ad088 Compare May 26, 2025 11:33
Copy link
Contributor

Note

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

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

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

@SvartMetal SvartMetal changed the title issue-3371: skip reporting of zeroblocks metrics issue-3371: skip reporting of ZeroBlocks metrics May 28, 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 667ad8b.

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

Copy link
Contributor

Note

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

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

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

@antonmyagkov antonmyagkov force-pushed the users/myagkov/issue-3371-skip-zero-blocks-metrics branch from 629b86f to 6a24f98 Compare May 30, 2025 19:48
Copy link
Contributor

Note

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants