test fail

qa/tasks/ceph_manager.py

@@ -1004,9 +1004,10 @@ def test_pool_min_size(self):
                                 break
                             self.log('not all PGs are active or peered')
                 else: # chose to revive OSDs, bring up a random fraction of the dead ones
-                    self.log("chose to revive osds")
-                    for i in range(1, int(rand_val * len(self.dead_osds))):
-                        self.revive_osd(i)
+                    self.log("chose to revive osds, dead osds are {dead}".format(dead=self.dead_osds))
+                    dead_osds_copy = self.dead_osds.copy()
+                    for osd in dead_osds_copy[:int(rand_val * len(dead_osds_copy))]:
+                        self.revive_osd(osd)
 
             # let PGs repair themselves or our next knockout might kill one
             self.ceph_manager.wait_for_clean(timeout=self.config.get('timeout'))
@@ -1246,7 +1247,9 @@ def choose_action(self):
                  (minin, minout, minlive, mindead, chance_down))
         actions = []
         if thrash_hosts:
+            self.log("check thrash_hosts")
             if len(self.in_osds) > minin:
+                self.log("check thrash_hosts: in_osds > minin")
                 actions.append((self.out_host, 1.0,))
         elif len(self.in_osds) > minin:
             actions.append((self.out_osd, 1.0,))