Skip to content
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

Implement EHistogramCounterOptions for THistogram class in storage #2503

Conversation

dvrazumov
Copy link
Collaborator

@dvrazumov dvrazumov commented Nov 14, 2024

Adding posibility to report histogram metrics in suitable for grafana format

  • Updated all THistogram counters (nrd/mirror3 storage) to support ReportHistogramAsMultipleCounters and ReportHistogramAsSingleCounter options
  • Propagated these options from config
  • Added some tests

Copy link
Contributor

Note

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

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

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

@dvrazumov dvrazumov force-pushed the users/dvrazumov/single-or-multiple-counter-histogram-in-TDiskPerfData branch from 9a4625d to 2259b4d Compare December 13, 2024 11:44
Copy link
Contributor

Note

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

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

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

@dvrazumov dvrazumov force-pushed the users/dvrazumov/single-or-multiple-counter-histogram-in-TDiskPerfData branch from 2259b4d to 46bfc5c Compare December 13, 2024 14:17
@dvrazumov dvrazumov changed the title Added tests, added EHistogramCounterOption to THistogram Implement EHistogramCounterOptions for THistogram class in storage Dec 13, 2024
Copy link
Contributor

Note

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

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

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

Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6281 6280 0 1 0 0

Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6281 6280 0 1 0 0

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

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

@dvrazumov dvrazumov requested a review from sharpeye December 16, 2024 10:05
@dvrazumov dvrazumov marked this pull request as ready for review December 16, 2024 10:05
Copy link
Contributor

Note

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

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

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

Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6279 6277 0 2 0 0

sharpeye
sharpeye previously approved these changes Dec 18, 2024
cloud/blockstore/libs/storage/core/disk_counters.h Outdated Show resolved Hide resolved
Copy link
Contributor

Note

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

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

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

sharpeye
sharpeye previously approved these changes Dec 19, 2024
@dvrazumov dvrazumov requested a review from drbasic December 19, 2024 12:23
@dvrazumov dvrazumov force-pushed the users/dvrazumov/single-or-multiple-counter-histogram-in-TDiskPerfData branch from bb7d7dd to 1cb0e8e Compare January 2, 2025 18:32
Copy link
Contributor

github-actions bot commented Jan 3, 2025

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5914 5912 0 2 0 0

Copy link
Contributor

github-actions bot commented Jan 3, 2025

Note

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

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

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

@dvrazumov dvrazumov requested a review from drbasic January 8, 2025 13:48
@dvrazumov dvrazumov force-pushed the users/dvrazumov/single-or-multiple-counter-histogram-in-TDiskPerfData branch from 8e0fea2 to 67973ca Compare January 9, 2025 15:40
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Note

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

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

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

Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6320 6312 0 7 1 0

@dvrazumov dvrazumov force-pushed the users/dvrazumov/single-or-multiple-counter-histogram-in-TDiskPerfData branch from e2839b9 to ab11400 Compare January 13, 2025 12:18
Copy link
Contributor

Note

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

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

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

sharpeye
sharpeye previously approved these changes Jan 14, 2025
Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6322 6320 0 2 0 0

@dvrazumov dvrazumov merged commit fa2a8d3 into main Jan 14, 2025
10 of 13 checks passed
@dvrazumov dvrazumov deleted the users/dvrazumov/single-or-multiple-counter-histogram-in-TDiskPerfData branch January 14, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[NBS, Filestore] Support suitable for Grafana data format for histograms counters
3 participants