]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/tasks: do not random.choice(a_view)
authorKefu Chai <kchai@redhat.com>
Thu, 2 Apr 2020 05:29:45 +0000 (13:29 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 7 Apr 2020 12:33:47 +0000 (20:33 +0800)
use `random.sample()` instead of `random.choice(list(a_view))` for better performance.

Signed-off-by: Kefu Chai <kchai@redhat.com>
qa/tasks/ceph_manager.py
qa/tasks/cephfs/test_dump_tree.py

index 3d41c2e3f482c4707bc85a6f6f83f1b54c0c8730..1ec5840d77bbba0b2b48c847bbf6492606592a7c 100644 (file)
@@ -1965,7 +1965,7 @@ class CephManager:
         """
         with self.lock:
             if self.pools:
-                return random.choice(self.pools.keys())
+                return random.sample(self.pools.keys(), 1)[0]
 
     def get_pool_pg_num(self, pool_name):
         """
index 6d943f9dd2ff337b13ac72cd84d8d8198833bb82..48a2c6f00acb55753f06b34ad48a128100334441 100644 (file)
@@ -39,7 +39,7 @@ class TestDumpTree(CephFSTestCase):
 
         self.populate()
         inos = self.get_paths_to_ino()
-        target = random.choice(inos.keys())
+        target = random.sample(inos.keys(), 1)[0]
 
         if target != "./":
             target = os.path.dirname(target)