Merge pull request #19208 from a6802739/TestSystemZoneConfigs

storage: fix bug for the testrace of TestSystemZoneConfigs
cockroachdb · Oct 12, 2017 · f0a3407 · f0a3407
2 parents e43f68f + 7fbd36c
commit f0a3407
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/storage/allocator.go b/pkg/storage/allocator.go
@@ -526,7 +526,11 @@ func (a Allocator) RebalanceTarget(
 			StoreID:   target.store.StoreID,
 			ReplicaID: rangeInfo.Desc.NextReplicaID,
 		}
+
+		// Deep-copy the Replicas slice since we'll mutate it.
 		desc := *rangeInfo.Desc
+		desc.Replicas = append([]roachpb.ReplicaDescriptor(nil), desc.Replicas...)
+
 		desc.Replicas = append(desc.Replicas, newReplica)
 		rangeInfo.Desc = &desc