]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: use small default pg count for CephFS pools 34817/head
authorPatrick Donnelly <pdonnell@redhat.com>
Wed, 9 Oct 2019 17:41:35 +0000 (10:41 -0700)
committerKefu Chai <kchai@redhat.com>
Wed, 29 Apr 2020 04:09:12 +0000 (12:09 +0800)
The pg count needs to be a power-of-two since
dff5697464edb9931d5dfa08cd4a30f85c1f237e.

Also, mon_pg_warn_min_per_osd is disabled by default now (or set to a
low value in vstart/testing) so there's no need to base the pg count on
this value.

Ideally someday we can remove this so that the default cluster value is
used but we need to keep this for deployments of older versions of Ceph.

Fixes: https://tracker.ceph.com/issues/42228
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit fc88e6c6c55402120a432ea47f05f321ba4c9bb1)

Conflicts:
qa/tasks/cephfs/filesystem.py: this commit was orignally
backported by #34055, but it failed to cherry-pick all necessary
bits. in this change, the missing bit is picked up.

qa/tasks/cephfs/filesystem.py

index 6667863486a339bcf05a8c8069baf9d56b0005a5..09dfc121af28c0cd955e042793d50d53861a34fd 100644 (file)
@@ -635,7 +635,7 @@ class Filesystem(MDSCluster):
 
     def add_data_pool(self, name, create=True):
         if create:
-            self.mon_manager.raw_cluster_cmd('osd', 'pool', 'create', name, self.pgs_per_fs_pool().__str__())
+            self.mon_manager.raw_cluster_cmd('osd', 'pool', 'create', name, self.pgs_per_fs_pool.__str__())
         self.mon_manager.raw_cluster_cmd('fs', 'add_data_pool', self.name, name)
         self.get_pool_names(refresh = True)
         for poolid, fs_name in self.data_pools.items():