Casey Bodley

@Zhiwei-Dai thanks for the contribution, this is a great feature! i'm seeing some test failures though, could you please take a look? this is with a vstart cluster with rgw...

for automated test coverage, we'll want to enable compression in rgw's encryption suite: ```diff diff --git a/qa/suites/rgw/crypt/3-rgw/rgw.yaml b/qa/suites/rgw/crypt/3-rgw/rgw.yaml index ee8d62af05c..764d216aade 100644 --- a/qa/suites/rgw/crypt/3-rgw/rgw.yaml +++ b/qa/suites/rgw/crypt/3-rgw/rgw.yaml @@ -6,6 +6,8 @@ overrides:...

passed qa in https://pulpito.ceph.com/cbodley-2022-10-19_23:28:37-rgw-wip-cbodley-testing-distro-default-smithi/ https://pulpito.ceph.com/cbodley-2022-10-20_13:42:11-rgw:crypt-wip-cbodley-testing-distro-default-smithi/

thanks @Zhiwei-Dai!

i've opened https://github.com/ceph/ceph/pull/48609 to document this new feature in PendingReleaseNotes

cc @yehudasa revisiting the description from https://github.com/ceph/ceph/pull/42710, i think we need to run the `ceph rgw realm bootstrap` part only once to create the original realm/zonegroup/zone additional zones need to...

> isn't the neorados code currently used with the RGW? shouldn't the neorados library support both a spawn/yield API and c++20 coroutines API? it does support both as part of...

> This looks mostly reasonable, I think? I'm not as familiar with how much background failure RBD has. > > https://pulpito.ceph.com/aemerson-2022-09-20_03:59:09-rbd-wip-coro-test-distro-default-smithi/ yikes, from those `cli_generic.sh` failures i see that `rbd...

@adamemerson i've been watching other runs of the rbd suite like https://pulpito.ceph.com/dis-2022-10-05_10:59:41-rbd-wip-dis-testing-distro-default-smithi/ and haven't seen any of those cli_generic.sh failures. you might try a rebase and rerun to see if...

looks much better! for the one test_librbd.sh failure, it got stuck on TestLibRBD.ConcurentOperations and never finished. not sure what's going on there, but maybe that's related to https://tracker.ceph.com/issues/53440? cc @idryomov