rgw.py: do not check for missing pools

oms4suse / python-ceph-cfg

Simple API to help deploying ceph.

Other

7 stars 12 forks source link

rgw.py: do not check for missing pools #6

Closed hoonetorg closed 8 years ago

hoonetorg commented 8 years ago

A few reasons for this change:

RGW pools have different names on Jewel.
these pools are autocreated in Jewel.
Creating pools which are not used for the 2 reasons above only increases pg's/pgp's per osd.
Multisite/Multizone does probably not need these standard rgw pools.

With this commit the module becomes more flexible. The user can still decide to run rgw_pools_create if he wants to create the pre-Jewel-standard rgw-pools.

Signed-off-by: Holger Fischer holger.fischer@hoonet.org

oms4suse commented 8 years ago

I like this, but think the pool check should be done on older version of ceph, I would be happy if this change was made, but first we need to get the ceph version and if its older than 0.81 then we can skip the test.

oms4suse commented 8 years ago

I have changed my mind, that can be a different commit later