testcluster: don't create two stoppers in StartTestCluster

This looks harmless because the first stopper was never actually used, but it was caught by cockroachdb#51413.
nvanbenschoten · Jul 22, 2020 · 83dfef8 · 83dfef8
1 parent 89dda79
commit 83dfef8
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/pkg/testutils/testcluster/testcluster.go b/pkg/testutils/testcluster/testcluster.go
@@ -141,7 +141,6 @@ func StartTestCluster(t testing.TB, nodes int, args base.TestClusterArgs) *TestC
 		stopper:         stop.NewStopper(),
 		replicationMode: args.ReplicationMode,
 	}
-	tc.stopper = stop.NewStopper()
 
 	// Check if any of the args have a locality set.
 	noLocalities := true