Skip to content

Commit

Permalink
roachtest: disable 23.1 -> 23.2 testing for follower reads
Browse files Browse the repository at this point in the history
After introducing shared process testing of follower reads, the tests
follower-reads/mixed-version/* have been flaky. This commit starts
testing from 23.2 to make sure that any other failures are caught and
triaged correctly.

This test is flaky when upgrading from v23.1 to v23.2 for follower reads
in shared-process deployments. There were a number of changes to tenant
health checks since then which appear to have addressed this issue.

Epic: none
Fixes: cockroachdb#133000
Fixes: cockroachdb#132999
Fixes: cockroachdb#129546
Fixes: cockroachdb#129167

Release note: None
  • Loading branch information
andrewbaptist authored and miraradeva committed Dec 11, 2024
1 parent 8bf9416 commit ff530bf
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions pkg/cmd/roachtest/tests/follower_reads.go
Original file line number Diff line number Diff line change
Expand Up @@ -1000,6 +1000,12 @@ func runFollowerReadsMixedVersionGlobalTableTest(
// Use a longer upgrade timeout to give the migrations enough time to finish
// considering the cross-region latency.
mixedversion.UpgradeTimeout(60*time.Minute),

// This test is flaky when upgrading from v23.1 to v23.2 for follower
// reads in shared-process deployments. There were a number of changes
// to tenant health checks since then which appear to have addressed
// this issue.
mixedversion.MinimumSupportedVersion("v23.2.0"),
)
}

Expand Down

0 comments on commit ff530bf

Please sign in to comment.