Skip to content

roachtest: widen disk-bandwidth-limiter lower-bound threshold#169748

Merged
trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
angeladietz:fix-disk-bw-limiter-threshold
May 5, 2026
Merged

roachtest: widen disk-bandwidth-limiter lower-bound threshold#169748
trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
angeladietz:fix-disk-bw-limiter-threshold

Conversation

@angeladietz
Copy link
Copy Markdown
Contributor

The lower-bound threshold for mean total bandwidth was too tight at 20%
(51.2 MiB/s), causing flaky failures across multiple platforms (GCE
amd64, GCE ARM64, IBM s390x) when bandwidth dipped marginally below the
threshold. Widen the margin from 20% to 30% (44.8 MiB/s), matching the
upper-bound margin. The limiter is working correctly in all observed
failures; the test was just too sensitive to normal I/O variability.

Fixes: #169721
Epic: none
Release note: None

The lower-bound threshold for mean total bandwidth was too tight at 20%
(51.2 MiB/s), causing flaky failures across multiple platforms (GCE
amd64, GCE ARM64, IBM s390x) when bandwidth dipped marginally below the
threshold. Widen the margin from 20% to 30% (44.8 MiB/s), matching the
upper-bound margin. The limiter is working correctly in all observed
failures; the test was just too sensitive to normal I/O variability.

Fixes: cockroachdb#169721
Epic: none
Release note: None

Co-Authored-By: roachdev-claude <roachdev-claude-bot@cockroachlabs.com>
@trunk-io
Copy link
Copy Markdown
Contributor

trunk-io Bot commented May 5, 2026

😎 Merged successfully - details.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@angeladietz angeladietz added the backport-26.2.x Flags PRs that need to be backported to 26.2 label May 5, 2026
@angeladietz angeladietz marked this pull request as ready for review May 5, 2026 14:38
@angeladietz angeladietz requested a review from a team May 5, 2026 14:38
@blathers-crl
Copy link
Copy Markdown

blathers-crl Bot commented May 5, 2026

Detected infrastructure failure (matched: self-hosted runner lost communication with the server). Automatically rerunning failed jobs. (run link)

@angeladietz
Copy link
Copy Markdown
Contributor Author

/trunk merge

@trunk-io trunk-io Bot merged commit 8b2bdce into cockroachdb:master May 5, 2026
38 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-26.2.x Flags PRs that need to be backported to 26.2 target-release-26.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

release-26.2: roachtest: admission-control/disk-bandwidth-limiter failed

3 participants