Skip to content

Conversation

@scylladbbot
Copy link

Adding new SCT utility to enhance the analysis of HDR files by running build_histograms_summary_with_interval with a shorter interval. This will help identify the specific process causing high latency

Testing

Example of result:

image

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Reminders

  • Add New configuration option and document them (in sdcm/sct_config.py)

  • Add unit tests to cover my changes (under unit-test/ folder)

  • Update the Readme/doc folder relevant to this change (if needed)

  • (cherry picked from commit 968cb0d)

  • (cherry picked from commit 0f38802)

Parent PR: #12154

@scylladbbot
Copy link
Author

@juliayakovlev - This PR has conflicts, therefore it was moved to draft
Please resolve them and mark this PR as ready for review

@scylladbbot scylladbbot force-pushed the backport/12154/to-perf-v17 branch from a5789df to 026348b Compare October 27, 2025 21:12
Adding new SCT utility to enhance the analysis of HDR files by running
build_histograms_summary_with_interval with a shorter interval.
This will help identify the specific process causing high latency

(cherry picked from commit 968cb0d)
Handle case when interval is larger than the time range in histogram summary by interval.
If the specified interval is greater than the difference between end_time and start_time,
use the actual time range as the window step to avoid empty intervals.

(cherry picked from commit 0f38802)
@fruch fruch force-pushed the backport/12154/to-perf-v17 branch from 026348b to f157ebf Compare October 28, 2025 13:43
@fruch fruch removed the conflicts label Oct 28, 2025
@fruch fruch marked this pull request as ready for review October 28, 2025 13:44
@fruch fruch merged commit c7d7d44 into scylladb:branch-perf-v17 Oct 28, 2025
4 checks passed
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.

3 participants