20240: Update test to reveal bug.
authorTom Clegg <tom@curii.com>
Thu, 23 Mar 2023 21:40:30 +0000 (17:40 -0400)
committerTom Clegg <tom@curii.com>
Tue, 4 Apr 2023 15:08:24 +0000 (11:08 -0400)
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

lib/controller/localdb/container_test.go

index 437e30b144cac8f12ca34f3a3e5d310b684d005b..5458a745d4dd5387fbdd2a6618c0a08aa0650624 100644 (file)
@@ -125,7 +125,14 @@ func (s *containerSuite) TestUpdatePriorityMultiLevelWorkflow(c *C) {
        // Set priority=0 on a parent+child, plus 18 other randomly
        // selected containers in the tree
        adminCtx := ctrlctx.NewWithToken(testCtx, s.cluster, s.cluster.SystemRootToken)
-       needfix := make([]int, 20)
+       // First entries of needfix are allcrs[1] (which is "i 0") and
+       // allcrs[2] ("i 0 j 0") -- we want to make sure to get at
+       // least one parent/child pair -- and the rest were chosen
+       // randomly.
+       //
+       // Similar randomly chosen sets (e.g., skipping 23 here) are
+       // known to fail. Possibly related to #20240.
+       needfix := []int{1, 2, 12, 20, 14, 13, 15, 7, 17, 28, 6, 26, 22, 21, 11, 1, 17, 18, 5}
        running := make(map[int]bool)
        for n := range needfix {
                var i int // which container are we going to run & then set priority=0