Skip to content

Commit

Permalink
roachtest: disable backfill success check
Browse files Browse the repository at this point in the history
The perturbation/*/backfill tests are flaky and are failing at least
once a week with the default configuration. This change temporarily
disables the check to allow easier investigation of the other failure
modes such as backfill failing to complete and node OOMs. Once those are
closed, and the test is running more stably, this threshold can be
dropped.

Fixes: cockroachdb#137093
Fixes: cockroachdb#137392

Informs: cockroachdb#133114

Release note: None
  • Loading branch information
andrewbaptist committed Jan 16, 2025
1 parent 0d61959 commit b8aabd0
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion pkg/cmd/roachtest/tests/perturbation/index_backfill.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ package perturbation
import (
"context"
"fmt"
"math"
"math/rand"
"time"

Expand All @@ -27,7 +28,9 @@ type backfill struct{}
var _ perturbation = backfill{}

func (b backfill) setup() variations {
v := setup(b, 40.0)
// TODO(baptist): Track down why this test causes stalls and drop the value
// to something more reasonable (like 5) once this is done.
v := setup(b, math.Inf(1))
return v
}

Expand Down

0 comments on commit b8aabd0

Please sign in to comment.