I was not able to run the tests as this seems to timeout for me, even after attempting to increase the timeout:

+ timeout 300 sh -c 'until [ $(ceph -s | grep -c "rgw:") -eq 1 ]; do echo "waiting for rgw to show up" && sleep 1; done'
waiting for rgw to show up
make: *** [Makefile:147: test-container] Error 124

CI has already found an error in the code, looks like you forgot to import C in the file.

# [github.com/ceph/go-ceph/rbd]
vet: rbd/data_pool_id.go:15:15: undefined: C
*** ERROR: returned 1

an example:

Process wise you are on track w/ the preview API stuff, thanks!

A nitpick about the commit message. Here's how the patch looks in my terminal:

commit 3c153860f9ee4d2197438ca3407d1b89ea1fe8f2 (HEAD -> pr.1164)
Author: Champ-Goblem <cameron@northflank.com>
Date:   Fri Aug 22 18:04:11 2025 +0100

    rbd: implement `rbd_get_data_pool_id`
    Add go-ceph binding for `rbd_get_data_pool_id` which is required to fetch the data pool for erasure coded volumes
    
    Signed-off-by: Champ-Goblem <cameron@northflank.com>

I'd request not to include backticks in the subject line as they introduce a 'markdownism' that is not needed in plain text and make typing the subject a little bit harder. Also, ideally there's a blank line between the subject and the next line. Also also, ideally a signed-off-by includes one's real name, some projects enforce it but we generally don't chase after people for it.

I have addressed your comments @phlogistonjohn, and apologies for the missing imports.
Let me know if anything else needs to be fixed!

I pushed a potential fix yesterday for the failing test. Hopefully that fixes it.