search

canonical / charm-microceph

Charm to deploy/manage microceph
Apache License 2.0
2 stars 9 forks source link

Implement additional broker operations #27

Closed lmlg closed 5 months ago

lmlg commented 6 months ago

This PR implements most of the broker operations missing in charm-microceph (compared to what's in charms.ceph). The only exceptions so far are those related to cache-tiering (because they need additional classes from charmhelpers). Additionally, most of the error checking stuff has been removed and delegated to the external calls. Note that most of the operations that call out to 'radosgw-admin' will not work unless RadosGW is installed, but they can be placeholders until we know what to do with these.

lmlg commented 6 months ago

Hi @lmlg this looks awesome. I wonder if we could have some testing for this, either unit or functional?

Yes, I forgot to mention: Let's not merge this until we have proper testing since this is a huge change LOC wise.