Skip to content

Commit

Permalink
roachtest: fix logic for exact replication
Browse files Browse the repository at this point in the history
Previously the check for exact replication was incorrectly counting the
number of replicas that were NOT at the desired replication count. This
fortunately worked becase it was a no-op in the only place that used it
since it was run immediately when all replicas had only a single
replica.

This correctly fixes the check.

Epic: none
Fixes: #109905
Fixes: #109906

Release note: None
  • Loading branch information
andrewbaptist authored and nvanbenschoten committed Mar 12, 2024
1 parent e0cb7ed commit d64bed3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/tests/util.go
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ func WaitForReplication(
var compStr string
switch waitForReplicationType {
case exactlyReplicationFactor:
compStr = "="
compStr = "!="
case atLeastReplicationFactor:
compStr = "<"
default:
Expand Down

0 comments on commit d64bed3

Please sign in to comment.