Correct unavailable spaces

don't create a [0, -1] case
basho · Oct 27, 2021 · 12cccff · 12cccff
1 parent 91fb709
commit 12cccff
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/riak_core_coverage_plan.erl b/src/riak_core_coverage_plan.erl
@@ -840,7 +840,13 @@ multi_failure_tester(PartitionCount, CoverageFun) when PartitionCount >= 32 ->
     % Fail two vnodes together - now can only get partial coverage from a r=2
     % plan
     RVN = rand:uniform(PartitionCount),
-    UnavailableKeySpaces1 = [RVN - 1, (RVN - 2) rem PartitionCount],
+    UnavailableKeySpaces1 = 
+        case RVN of
+            1 ->
+                [0, PartitionCount - 1];
+            _ ->
+                [RVN - 1, RVN - 2]
+        end,
     {insufficient_vnodes_available, _, VnodeCovers1} =
         CoverageFun(ReqId, NVal, PartitionCount,
                     UnavailableKeySpaces1, PVC),