From 9bd14db5f459e3f9bf11ce0c5ba6ead3e0d221ee Mon Sep 17 00:00:00 2001 From: Raphael 'kena' Poss Date: Mon, 31 Aug 2020 20:25:02 +0200 Subject: [PATCH] roachtest: re-enable acceptance/bank/zerosum-splits PR #34080 which intended to skip `bank/zerosum-restarts`, mistakenly skipped `bank/zerosum-splits` instead. (As `zerosum-restarts` had been skipped already prior) There was no reason to skip `zerosum-splits`, so this commit re-enables it. Release justification: non-production code changes Release note: None --- pkg/cmd/roachtest/acceptance.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/pkg/cmd/roachtest/acceptance.go b/pkg/cmd/roachtest/acceptance.go index a89dd1036f07..61f919347622 100644 --- a/pkg/cmd/roachtest/acceptance.go +++ b/pkg/cmd/roachtest/acceptance.go @@ -26,11 +26,7 @@ func registerAcceptance(r *testRegistry) { // Sorted. Please keep it that way. {name: "bank/cluster-recovery", fn: runBankClusterRecovery}, {name: "bank/node-restart", fn: runBankNodeRestart}, - { - name: "bank/zerosum-splits", fn: runBankNodeZeroSum, - skip: "https://github.com/cockroachdb/cockroach/issues/33683 (runs into " + - " various errors during its rebalances, see IsExpectedRelocateError)", - }, + {name: "bank/zerosum-splits", fn: runBankNodeZeroSum}, {name: "bank/zerosum-restart", fn: runBankZeroSumRestart}, {name: "build-info", fn: runBuildInfo}, {name: "build-analyze", fn: runBuildAnalyze},