Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
118311: roachtest: use latest predecessor version in change repls r=erikgrinaker a=kvoli

`change-replicas/mixed-version` could run into a now fixed etcd raft bug where replicas never exited `StateProbe` resolved by cockroachdb#106748. Only use the latest predecessor version to avoid re-diagnosing this bug.

Fixes: cockroachdb#118006
Release note: None

Co-authored-by: Austen McClernon <[email protected]>
  • Loading branch information
craig[bot] and kvoli committed Jan 29, 2024
2 parents d305898 + bdd74be commit a68bc94
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion pkg/cmd/roachtest/tests/mixed_version_change_replicas.go
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,12 @@ func runChangeReplicasMixedVersion(ctx context.Context, t test.Test, c cluster.C

// Set up and run test.
mvt := mixedversion.NewTest(ctx, t, t.L(), c, c.All(), mixedversion.ClusterSettingOption(
install.EnvOption{"COCKROACH_SCAN_MAX_IDLE_TIME=10ms"})) // speed up queues
// Speed up the queues.
install.EnvOption{"COCKROACH_SCAN_MAX_IDLE_TIME=10ms"}),
// Avoid repeatedly running into #114549 on earlier minor versions.
// TODO(kvoli): Remove in 24.2.
mixedversion.AlwaysUseLatestPredecessors,
)

mvt.OnStartup("create test table", createTable)
mvt.InMixedVersion("move replicas", moveReplicas)
Expand Down

0 comments on commit a68bc94

Please sign in to comment.