Skip to content

Commit

Permalink
acceptance: re-enable bank/zerosum-restart
Browse files Browse the repository at this point in the history
This was forgotten, #30064 is long fixed :fingers_crossed:

Release note: None
  • Loading branch information
tbg committed Jan 10, 2019
1 parent 871ed6a commit 1b3fe9c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions pkg/cmd/roachtest/acceptance.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,7 @@ func registerAcceptance(r *registry) {
{"bank/cluster-recovery", runBankClusterRecovery},
{"bank/node-restart", runBankNodeRestart},
{"bank/zerosum-splits", runBankNodeZeroSum},
//TODO(masha) This is flaking for same reasons as bank/node-restart
// enable after #30064 is fixed.
//{"bank/zerosum-restart", runBankZeroSumRestart},
{"bank/zerosum-restart", runBankZeroSumRestart},
{"build-info", runBuildInfo},
{"cli/node-status", runCLINodeStatus},
{"decommission", runDecommissionAcceptance},
Expand Down
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/bank.go
Original file line number Diff line number Diff line change
Expand Up @@ -345,7 +345,7 @@ func (s *bankState) startSplitMonkey(ctx context.Context, d time.Duration, c *cl
}

func isExpectedRelocateError(err error) bool {
return testutils.IsError(err, "(descriptor changed|unable to remove replica .* which is not present|unable to add replica .* which is already present|received invalid ChangeReplicasTrigger .* to remove self)")
return testutils.IsError(err, "(descriptor changed|unable to remove replica .* which is not present|unable to add replica .* which is already present|received invalid ChangeReplicasTrigger .* to remove self|breaker open)")
}

func accountDistribution(r *rand.Rand) *rand.Zipf {
Expand Down

0 comments on commit 1b3fe9c

Please sign in to comment.