From e955e063826b2368e9581bd892cdfd1cdfe4f8a3 Mon Sep 17 00:00:00 2001 From: "James E. Blair" Date: Mon, 8 Oct 2012 09:49:03 -0700 Subject: [PATCH] Fix non-deterministic test. I hope. Change-Id: I813fdeb9ecd2e6f64871bc9499b0d0919a9cea83 Reviewed-on: https://review.openstack.org/14171 Reviewed-by: Clark Boylan Approved: James E. Blair Tested-by: Jenkins --- tests/test_scheduler.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/tests/test_scheduler.py b/tests/test_scheduler.py index 9b621c4b38..8ba6594edc 100644 --- a/tests/test_scheduler.py +++ b/tests/test_scheduler.py @@ -1488,9 +1488,14 @@ class testScheduler(unittest.TestCase): for x in jobs: print x # all jobs running - jobs[0].release() - jobs[1].release() - jobs[2].release() + + # Grab pointers to the jobs we want to release before + # releasing any, because list indexes may change as + # the jobs complete. + a, b, c = jobs[:3] + a.release() + b.release() + c.release() self.waitUntilSettled() self.fake_jenkins.hold_jobs_in_build = False